mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-08-05 22:56:57 +02:00
Code Issues Projects Releases Wiki Activity

MINOR: hlua: Use the buffer instead of the HTTP message to get HTTP headers

Browse Source 
hlua_http_get_headers() function was using the HTTP message from the stream
TXN to retrieve headers from a message. However, this will be an issue to
update the lua HTTP applet to use its own buffers. Indeed, in that case,
information from the channels will be unavailable. So now,
hlua_http_get_headers() is now using a buffer containing an HTX message. It
is just an API change bacause, internally, the function was already
manipulation an HTX message.
This commit is contained in:
Christopher Faulet 2025-07-18 12:07:05 +02:00
parent 15080d9aae
commit 3e456be5ae
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/hlua.c
View File

@ -707,7 +707,7 @@ __LJMP static int hlua_smp2lua(lua_State *L, struct sample *smp);
__LJMP static int hlua_smp2lua_str(lua_State *L, struct sample *smp);
static int hlua_lua2smp(lua_State *L, int ud, struct sample *smp);
__LJMP static int hlua_http_get_headers(lua_State *L, struct http_msg *msg);
__LJMP static int hlua_http_get_headers(lua_State *L, struct buffer *buf);
struct prepend_path {
struct list l;
@ -6423,7 +6423,7 @@ __LJMP static int hlua_http_get_stline(lua_State *L, struct htx_sl *sl)
* This function does not fails. It is used as wrapper with the
* 2 following functions.
*/
__LJMP static int hlua_http_get_headers(lua_State *L, struct http_msg *msg)
__LJMP static int hlua_http_get_headers(lua_State *L, struct buffer *buf)
{
struct htx *htx;
int32_t pos;
@ -6494,7 +6494,7 @@ __LJMP static int hlua_http_req_get_headers(lua_State *L)
if (htxn->dir != SMP_OPT_DIR_REQ || !IS_HTX_STRM(htxn->s))
WILL_LJMP(lua_error(L));
return hlua_http_get_headers(L, &htxn->s->txn->req);
return hlua_http_get_headers(L, &htxn->s->req.buf);
}
__LJMP static int hlua_http_res_get_headers(lua_State *L)
@ -6507,7 +6507,7 @@ __LJMP static int hlua_http_res_get_headers(lua_State *L)
if (htxn->dir != SMP_OPT_DIR_RES || !IS_HTX_STRM(htxn->s))
WILL_LJMP(lua_error(L));
return hlua_http_get_headers(L, &htxn->s->txn->rsp);
return hlua_http_get_headers(L, &htxn->s->res.buf);
}
/* This function replace full header, or just a value in
@ -6898,7 +6898,7 @@ __LJMP static int hlua_http_msg_get_headers(lua_State *L)
if (msg->msg_state > HTTP_MSG_BODY)
WILL_LJMP(lua_error(L));
return hlua_http_get_headers(L, msg);
return hlua_http_get_headers(L, &msg->chn->buf);
}
/* Deletes all occurrences of an header in the HTTP message matching on its
@ -11355,7 +11355,7 @@ void hlua_applet_http_fct(struct appctx *ctx)
* the array on the top of the stack.
*/
lua_pushstring(hlua->T, "headers");
if (!hlua_http_get_headers(hlua->T, &strm->txn->req))
if (!hlua_http_get_headers(hlua->T, &strm->req.buf))
goto error;
lua_settable(hlua->T, app_idx - 3);