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_httpd.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_httpd.h')
| -rw-r--r-- | xs_httpd.h | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -36,7 +36,7 @@ xs_dict *xs_httpd_request(FILE *f, xs_str **payload, int *p_size) { /* split the path with its optional variables */ - xs *udp = xs_url_dec(xs_list_get(l2, 1)); + const xs_val *udp = xs_list_get(l2, 1); xs *pnv = xs_split_n(udp, "?", 1); /* store the path */ @@ -75,8 +75,7 @@ xs_dict *xs_httpd_request(FILE *f, xs_str **payload, int *p_size) v = xs_dict_get(req, "content-type"); if (*payload && v && strcmp(v, "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 && v && xs_startswith(v, "multipart/form-data")) { |