Mark, On 1/27/17 4:02 PM, ma...@apache.org wrote: > Author: markt > Date: Fri Jan 27 21:02:08 2017 > New Revision: 1780609 > > URL: http://svn.apache.org/viewvc?rev=1780609&view=rev > Log: > Follow-up to r1780606 > Ensure RFC2109 cookie is created with correct version > > Modified: > tomcat/trunk/java/org/apache/tomcat/util/http/parser/Cookie.java > tomcat/trunk/test/org/apache/tomcat/util/http/TestCookies.java > > Modified: tomcat/trunk/java/org/apache/tomcat/util/http/parser/Cookie.java > URL: > http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/http/parser/Cookie.java?rev=1780609&r1=1780608&r2=1780609&view=diff > ============================================================================== > --- tomcat/trunk/java/org/apache/tomcat/util/http/parser/Cookie.java > (original) > +++ tomcat/trunk/java/org/apache/tomcat/util/http/parser/Cookie.java Fri Jan > 27 21:02:08 2017 > @@ -145,7 +145,7 @@ public class Cookie { > skipLWS(bb); > byte b = bb.get(); > if (b == SEMICOLON_BYTE || b == COMMA_BYTE) { > - parseCookieRfc2109(bb, serverCookies); > + parseCookieRfc2109(bb, serverCookies, version - 48);
Would it be more self-documenting to subtract '0' instead of 48, there? -chris
signature.asc
Description: OpenPGP digital signature