[PATCH] Allow URLs with IPv6 literals in AnyEvent::HTTP

Marc Lehmann schmorp at schmorp.de
Sun May 10 08:20:00 CEST 2015


On Sat, May 09, 2015 at 12:29:48PM +0200, Raphael Geissert <geissert at debian.org> wrote:
> On 8 May 2015 at 23:26, Marc Lehmann <schmorp at schmorp.de> wrote:
> > On Sun, May 03, 2015 at 03:18:23PM +0200, Raphael Geissert <geissert at debian.org> wrote:
> >> Attached patch (with test) fixes AnyEvent::HTTP's parsing of URLs so
> >> that it accepts IPv6 literals.
> >
> > Thanks, I solved it quite a bit differently (the problem was not missing
> > support but simply a parsing bug, as the code to parse ipv6 literals is
> > already there), but the patch and the test were very useful!
> 
> In that case, does it still make sense to use [^@]  instead of a
> non-greedy dot-plus match?

Hmm, why not?

> It looks good otherwise.

Great :)

-- 
                The choice of a       Deliantra, the free code+content MORPG
      -----==-     _GNU_              http://www.deliantra.net
      ----==-- _       generation
      ---==---(_)__  __ ____  __      Marc Lehmann
      --==---/ / _ \/ // /\ \/ /      schmorp at schmorp.de
      -=====/_/_//_/\_,_/ /_/\_\



More information about the anyevent mailing list