Commit63438e9

committed

Fix#499 - Parsing empty query parameters

1 parentb5f6838 commit63438e9Copy full SHA for 63438e9

File tree

+14

-1

lines changed

+14

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ struct key_value_sequence : spirit::qi::grammar<uri::const_iterator, Map()> {`
`27`	`27`	`pair = key >> -('=' >> value);`
`28`	`28`	`key =`
`29`	`29`	`spirit::qi::char_("a-zA-Z_") >> *spirit::qi::char_("-+.~a-zA-Z_0-9/%");`
`30`		`- value =+spirit::qi::char_("-+.~a-zA-Z_0-9/%");`
	`30`	`+ value =*spirit::qi::char_("-+.~a-zA-Z_0-9/%");`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`spirit::qi::rule<uri::const_iterator, Map()> query;`

Lines changed: 13 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -566,3 +566,16 @@ BOOST_AUTO_TEST_CASE(issue_447_test) {`
`566`	`566`	`uri::uriinstance("http://[www.foo.com/");`
`567`	`567`	`BOOST_REQUIRE(!uri::valid(instance));`
`568`	`568`	`}`
	`569`	`+`
	`570`	`+BOOST_AUTO_TEST_CASE(issue_499_test) {`
	`571`	`+ uri::uriinstance(`
	`572`	`+"http://www.example.com/path?param1&param2=&param3=value");`
	`573`	`+BOOST_REQUIRE(uri::valid(instance));`
	`574`	`+`
	`575`	`+ std::map<std::string, std::string> queries;`
	`576`	`+uri::query_map(instance, queries);`
	`577`	`+BOOST_REQUIRE_EQUAL(queries.size(),std::size_t(3));`
	`578`	`+BOOST_CHECK_EQUAL(queries["param1"],"");`
	`579`	`+BOOST_CHECK_EQUAL(queries["param2"],"");`
	`580`	`+BOOST_CHECK_EQUAL(queries["param3"],"value");`
	`581`	`+}`

Comments

(0)