Skip to content

Commit

Permalink
Merge branch 'tr/maint-no-unquote-plus'
Browse files Browse the repository at this point in the history
* tr/maint-no-unquote-plus:
  Do not unquote + into ' ' in URLs
  • Loading branch information
Junio C Hamano committed Aug 31, 2010
2 parents 633142d + 730220d commit d7cc7c9
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 7 deletions.
10 changes: 8 additions & 2 deletions t/t5601-clone.sh
Original file line number Diff line number Diff line change
Expand Up @@ -178,8 +178,14 @@ test_expect_success 'clone respects global branch.autosetuprebase' '

test_expect_success 'respect url-encoding of file://' '
git init x+y &&
test_must_fail git clone "file://$PWD/x+y" xy-url &&
git clone "file://$PWD/x%2By" xy-url
git clone "file://$PWD/x+y" xy-url-1 &&
git clone "file://$PWD/x%2By" xy-url-2
'

test_expect_success 'do not query-string-decode + in URLs' '
rm -rf x+y &&
git init "x y" &&
test_must_fail git clone "file://$PWD/x+y" xy-no-plus
'

test_expect_success 'do not respect url-encoding of non-url path' '
Expand Down
11 changes: 6 additions & 5 deletions url.c
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,8 @@ static int url_decode_char(const char *q)
return val;
}

static char *url_decode_internal(const char **query, const char *stop_at, struct strbuf *out)
static char *url_decode_internal(const char **query, const char *stop_at,
struct strbuf *out, int decode_plus)
{
const char *q = *query;

Expand All @@ -90,7 +91,7 @@ static char *url_decode_internal(const char **query, const char *stop_at, struct
}
}

if (c == '+')
if (decode_plus && c == '+')
strbuf_addch(out, ' ');
else
strbuf_addch(out, c);
Expand All @@ -110,17 +111,17 @@ char *url_decode(const char *url)
strbuf_add(&out, url, colon - url);
url = colon;
}
return url_decode_internal(&url, NULL, &out);
return url_decode_internal(&url, NULL, &out, 0);
}

char *url_decode_parameter_name(const char **query)
{
struct strbuf out = STRBUF_INIT;
return url_decode_internal(query, "&=", &out);
return url_decode_internal(query, "&=", &out, 1);
}

char *url_decode_parameter_value(const char **query)
{
struct strbuf out = STRBUF_INIT;
return url_decode_internal(query, "&", &out);
return url_decode_internal(query, "&", &out, 1);
}

0 comments on commit d7cc7c9

Please sign in to comment.