diff options
| author | Nicolai Dagestad <nicolai@dagestad.fr> | 2024-09-15 15:03:21 +0200 |
|---|---|---|
| committer | Nicolai Dagestad <nicolai@dagestad.fr> | 2024-09-15 18:39:49 +0200 |
| commit | ba5cbb6d828165a43826c6afdd71fa2edbdca302 (patch) | |
| tree | 87290e2f2a97433c35f0ce86cd14cabe9703dd2c /xs_fcgi.h | |
| parent | 31ce1af73630143036d9cfc6a8a5083402f6b7aa (diff) | |
URL decode data after splitting the arguments
Data decoding should happen after the parsing if not, a '?', '&', '#'
or other character decoded will interfere with the parsing. e.g. the
users password contains a '&', then it is truncated on that character,
and login will fail.
Diffstat (limited to 'xs_fcgi.h')
| -rw-r--r-- | xs_fcgi.h | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -179,8 +179,7 @@ xs_dict *xs_fcgi_request(FILE *f, xs_str **payload, int *p_size, int *fcgi_id) req = xs_dict_append(req, "method", v); else if (strcmp(k, "REQUEST_URI") == 0) { - xs *udp = xs_url_dec(v); - xs *pnv = xs_split_n(udp, "?", 1); + xs *pnv = xs_split_n(v, "?", 1); /* store the path */ req = xs_dict_append(req, "path", xs_list_get(pnv, 0)); @@ -233,8 +232,7 @@ xs_dict *xs_fcgi_request(FILE *f, xs_str **payload, int *p_size, int *fcgi_id) const char *ct = xs_dict_get(req, "content-type"); if (*payload && ct && strcmp(ct, "application/x-www-form-urlencoded") == 0) { - xs *upl = xs_url_dec(*payload); - p_vars = xs_url_vars(upl); + p_vars = xs_url_vars(*payload); } else if (*payload && ct && xs_startswith(ct, "multipart/form-data")) { |