wget: --post-data=STR should not encode STR, should send it verbatim
This matches GNU Wget 1.12 behavior. Signed-off-by: Vitaly Magerya <vmagerya@gmail.com> Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
parent
a9e5c43b8b
commit
700fbc308d
@ -344,44 +344,6 @@ static char *gethdr(FILE *fp)
|
||||
return hdrval;
|
||||
}
|
||||
|
||||
#if ENABLE_FEATURE_WGET_LONG_OPTIONS
|
||||
static char *URL_escape(const char *str)
|
||||
{
|
||||
/* URL encode, see RFC 2396 */
|
||||
char *dst;
|
||||
char *res = dst = xmalloc(strlen(str) * 3 + 1);
|
||||
unsigned char c;
|
||||
|
||||
while (1) {
|
||||
c = *str++;
|
||||
if (c == '\0'
|
||||
/* || strchr("!&'()*-.=_~", c) - more code */
|
||||
|| c == '!'
|
||||
|| c == '&'
|
||||
|| c == '\''
|
||||
|| c == '('
|
||||
|| c == ')'
|
||||
|| c == '*'
|
||||
|| c == '-'
|
||||
|| c == '.'
|
||||
|| c == '='
|
||||
|| c == '_'
|
||||
|| c == '~'
|
||||
|| (c >= '0' && c <= '9')
|
||||
|| ((c|0x20) >= 'a' && (c|0x20) <= 'z')
|
||||
) {
|
||||
*dst++ = c;
|
||||
if (c == '\0')
|
||||
return res;
|
||||
} else {
|
||||
*dst++ = '%';
|
||||
*dst++ = bb_hexdigits_upcase[c >> 4];
|
||||
*dst++ = bb_hexdigits_upcase[c & 0xf];
|
||||
}
|
||||
}
|
||||
}
|
||||
#endif
|
||||
|
||||
static FILE* prepare_ftp_session(FILE **dfpp, struct host_info *target, len_and_sockaddr *lsa)
|
||||
{
|
||||
FILE *sfp;
|
||||
@ -716,15 +678,13 @@ static void download_one_url(const char *url)
|
||||
fputs(G.extra_headers, sfp);
|
||||
|
||||
if (option_mask32 & WGET_OPT_POST_DATA) {
|
||||
char *estr = URL_escape(G.post_data);
|
||||
fprintf(sfp,
|
||||
"Content-Type: application/x-www-form-urlencoded\r\n"
|
||||
"Content-Length: %u\r\n"
|
||||
"\r\n"
|
||||
"%s",
|
||||
(int) strlen(estr), estr
|
||||
(int) strlen(G.post_data), G.post_data
|
||||
);
|
||||
free(estr);
|
||||
} else
|
||||
#endif
|
||||
{
|
||||
|
Loading…
Reference in New Issue
Block a user