andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #73 from ngocdaothanh/master

Browse Source 
QueryStringDecoder should also support ; Fix for #72
This commit is contained in:
Norman Maurer 2011-11-22 22:08:31 -08:00
commit b5f8ff3003
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/main/java/org/jboss/netty/handler/codec/http/QueryStringDecoder.java
View File

@ -30,9 +30,11 @@ import org.jboss.netty.util.CharsetUtil;
* Splits an HTTP query string into a path string and key-value parameter pairs. * Splits an HTTP query string into a path string and key-value parameter pairs.
* This decoder is for one time use only. Create a new instance for each URI: * This decoder is for one time use only. Create a new instance for each URI:
* <pre> * <pre>
* {@link QueryStringDecoder} decoder = new {@link QueryStringDecoder}("/hello?recipient=world"); * {@link QueryStringDecoder} decoder = new {@link QueryStringDecoder}("/hello?recipient=world&x=1;y=2");
* assert decoder.getPath().equals("/hello"); * assert decoder.getPath().equals("/hello");
* assert decoder.getParameters().get("recipient").equals("world"); * assert decoder.getParameters().get("recipient").equals("world");
* assert decoder.getParameters().get("x").equals("1");
* assert decoder.getParameters().get("y").equals("2");
* </pre> * </pre>
* *
* @author <a href="http://www.jboss.org/netty/">The Netty Project</a> * @author <a href="http://www.jboss.org/netty/">The Netty Project</a>
@ -73,7 +75,8 @@ public class QueryStringDecoder {
throw new NullPointerException("charset"); throw new NullPointerException("charset");
} }
this.uri = uri; // http://en.wikipedia.org/wiki/Query_string
this.uri = uri.replace(';', '&');
this.charset = charset; this.charset = charset;
} }
@ -97,7 +100,8 @@ public class QueryStringDecoder {
throw new NullPointerException("charset"); throw new NullPointerException("charset");
} }
this.uri = uri.toASCIIString(); // http://en.wikipedia.org/wiki/Query_string
this.uri = uri.toASCIIString().replace(';', '&');
this.charset = charset; this.charset = charset;
} }