2010-02-03 Tatsuhiro Tsujikawa <t-tujikawa@users.sourceforge.net>

Remove fragment from redirected URI.
	* src/Request.cc
	* test/RequestTest.cc
pull/1/head
Tatsuhiro Tsujikawa 2010-02-03 13:49:22 +00:00
parent 6fa1dc11b2
commit 01fc3204f2
3 changed files with 9 additions and 3 deletions

View File

@ -1,3 +1,9 @@
2010-02-03 Tatsuhiro Tsujikawa <t-tujikawa@users.sourceforge.net>
Remove fragment from redirected URI.
* src/Request.cc
* test/RequestTest.cc
2010-02-03 Tatsuhiro Tsujikawa <t-tujikawa@users.sourceforge.net> 2010-02-03 Tatsuhiro Tsujikawa <t-tujikawa@users.sourceforge.net>
Updated po templates. Updated po templates.

View File

@ -141,7 +141,7 @@ bool Request::redirectUrl(const std::string& url) {
} else { } else {
redirectedUrl = url; redirectedUrl = url;
} }
return parseUrl(urlencode(redirectedUrl)); return parseUrl(urlencode(removeFragment(redirectedUrl)));
} }
bool Request::parseUrl(const std::string& url) { bool Request::parseUrl(const std::string& url) {

View File

@ -363,8 +363,8 @@ void RequestTest::testRedirectUrl() {
req.getCurrentUrl()); req.getCurrentUrl());
CPPUNIT_ASSERT_EQUAL((unsigned int)3, req.getRedirectCount()); CPPUNIT_ASSERT_EQUAL((unsigned int)3, req.getRedirectCount());
// White space in path // White space in path and fragment is appended.
CPPUNIT_ASSERT(req.redirectUrl("http://example.org/white space")); CPPUNIT_ASSERT(req.redirectUrl("http://example.org/white space#aria2"));
CPPUNIT_ASSERT_EQUAL(std::string("http://example.org/white%20space"), CPPUNIT_ASSERT_EQUAL(std::string("http://example.org/white%20space"),
req.getCurrentUrl()); req.getCurrentUrl());
} }