On Sun, 29 Apr 2018 at 17:10, Violeta Georgieva <violet...@apache.org> wrote:
> On Sun, 29 Apr 2018 at 6:54, Mark Thomas <ma...@apache.org> wrote: > >> On 28/04/18 17:50, Violeta Georgieva wrote: >> > Hi Mark, >> > >> > 2018-04-26 21:09 GMT+03:00 <ma...@apache.org>: >> >> >> >> Author: markt >> >> Date: Thu Apr 26 18:09:19 2018 >> >> New Revision: 1830251 >> >> >> >> URL: http://svn.apache.org/viewvc?rev=1830251&view=rev >> >> Log: >> >> Correct various edge cases in the new HTTP Host header validation >> parser. >> >> Patch provided by Katya Todorova. >> >> >> >> Fix IPv6/IPv4 parsing for host header: >> >> - chars other than : should not be allowed in IPv6 address after ] >> >> - ::: should not present in IPv6 address >> >> - IPv4 part of IPv6 address was not correctly parsed (1 symbol of >> > IPv4 part was ignored) >> >> - tests added to cover IPv4/6 parsing >> >> - parsed test class fixed not to throw NPE when an exception is >> > expected but not thrown >> >> >> >> Modified: >> >> tomcat/tc8.0.x/trunk/ (props changed) >> >> >> > >> tomcat/tc8.0.x/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java >> >> tomcat/tc8.0.x/trunk/res/maven/mvn-pub.xml >> > >> > Is this file really part of that change? >> >> No. I think it got caught up with that commit by mistake. I don't think >> it does any harm though. >> > > Only for the release ;) > If you don’t mind I’ll revert this file to its previous version. Wdyt? > >> Mark >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org >> For additional commands, e-mail: dev-h...@tomcat.apache.org >> >>