diff options
| author | Leonardo Taccari <iamleot@gmail.com> | 2018-08-23 23:27:34 +0200 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2018-08-24 08:57:49 +0200 |
| commit | 8f3c3cd08a5b252002a4abfb19780850fc51040e (patch) | |
| tree | 250835eb90c46584c1ba55bd88d40c4cd7008868 /lib/gopher.c | |
| parent | 387e85efd08d9a7d184c9fd37cba61110728e511 (diff) | |
| download | curl-8f3c3cd08a5b252002a4abfb19780850fc51040e.tar.gz | |
gopher: Do not translate `?' to `%09'
Since GOPHER support was added in curl `?' character was automatically
translated to `%09' (`\t').
However, this behaviour does not seems documented in RFC 4266 and for
search selectors it is documented to directly use `%09' in the URL.
Apart that several gopher servers in the current gopherspace have CGI
support where `?' is used as part of the selector and translating it to
`%09' often leads to surprising results.
Closes #2910
Diffstat (limited to 'lib/gopher.c')
| -rw-r--r-- | lib/gopher.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/lib/gopher.c b/lib/gopher.c index b7c31b695..d65049969 100644 --- a/lib/gopher.c +++ b/lib/gopher.c @@ -93,18 +93,11 @@ static CURLcode gopher_do(struct connectdata *conn, bool *done) } else { char *newp; - size_t j, i; /* Otherwise, drop / and the first character (i.e., item type) ... */ newp = path; newp += 2; - /* ... then turn ? into TAB for search servers, Veronica, etc. ... */ - j = strlen(newp); - for(i = 0; i<j; i++) - if(newp[i] == '?') - newp[i] = '\x09'; - /* ... and finally unescape */ result = Curl_urldecode(data, newp, 0, &sel, &len, FALSE); if(result) |