Mailing List Archive

HTTP::Body not storing raw bodies of UrlEncoded requests (RT #111876) - opinions requested
Hi folks,

I have a problem with HTTP::Body not returning the raw body content for
responses parsed by the UrlEncoded parser, which is causing us a
problem in Dancer1:

https://github.com/PerlDancer/Dancer/issues/1140

My preferred fix for this is for HTTP::Body::UrlEncoded to stash the
body being parsed away in a temp file as the other parsers already do -
which I've submitted as a patch in RT #111876:

https://rt.cpan.org/Ticket/Display.html?id=111876

First off - do you consider that sensible, and would you consider
applying it and getting a new HTTP::Body release out - and if so, would
that be likely to be doable in the near future? (If you agreed with
the change but don't have time to maintain HTTP::Body or get new
releases out, I may be willing to step up as a co-maint and clear some
of the older tickets in the queue too perhaps.)

Alternatively - if you don't agree with my proposed change, or don't
think a new release with it in could be forthcoming for some time,
would you have any objections to me bundling up my modified version of
HTTP::Body as Dancer::HTTP::Body and shipping it with Dancer
(maintaining the licencing/author information and with PAUSE told not
to index it)?

Cheers

Dave P (bigpresh)


_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev
Re: HTTP::Body not storing raw bodies of UrlEncoded requests (RT #111876) - opinions requested [ In reply to ]
David,

Not sure the devs for this monitor this mailing list. Maybe try IRC or emailing the last person to release directly?

jnap



On Monday, February 8, 2016 6:05 PM, David Precious <davidp@preshweb.co.uk> wrote:

Hi folks,

I have a problem with HTTP::Body not returning the raw body content for
responses parsed by the UrlEncoded parser, which is causing us a
problem in Dancer1:

https://github.com/PerlDancer/Dancer/issues/1140

My preferred fix for this is for HTTP::Body::UrlEncoded to stash the
body being parsed away in a temp file as the other parsers already do -
which I've submitted as a patch in RT #111876:

https://rt.cpan.org/Ticket/Display.html?id=111876

First off - do you consider that sensible, and would you consider
applying it and getting a new HTTP::Body release out - and if so, would
that be likely to be doable in the near future? (If you agreed with
the change but don't have time to maintain HTTP::Body or get new
releases out, I may be willing to step up as a co-maint and clear some
of the older tickets in the queue too perhaps.)

Alternatively - if you don't agree with my proposed change, or don't
think a new release with it in could be forthcoming for some time,
would you have any objections to me bundling up my modified version of
HTTP::Body as Dancer::HTTP::Body and shipping it with Dancer
(maintaining the licencing/author information and with PAUSE told not
to index it)?

Cheers

Dave P (bigpresh)


_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev

_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev
Re: HTTP::Body not storing raw bodies of UrlEncoded requests (RT #111876) - opinions requested [ In reply to ]
Hi John,

On Tue, 9 Feb 2016 01:30:07 +0000 (UTC) John Napiorkowski
<jjn1056@yahoo.com> wrote:
> > On Monday, February 8, 2016 6:05 PM, David Precious
> > <davidp@preshweb.co.uk> wrote:
> > I have a problem with HTTP::Body [...]

> David,
>
> Not sure the devs for this monitor this mailing list. Maybe try IRC
> or emailing the last person to release directly?

Ah, I was going by the HTTP::Body docs which suggested this list as
point of contact. If I get no further responses, I'll jump on IRC or
attempt to poke GETTY directly.

Cheers!


_______________________________________________
Catalyst-dev mailing list
Catalyst-dev@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst-dev