Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI CPAN distribution.

Report information
The Basics
Id: 79102
Status: resolved
Priority: 0/
Queue: CGI

People
Owner: Nobody in particular
Requestors:
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



Subject: PUTDATA/POSTDATA, uncorrupted by -utf8, and as tempfiles
inspired by http://stackoverflow.com/questions/11343663 use CGI -utf8; would/could corrupt PUTDATA/POSTDATA -- fixed I also add -putdata_upload to tread param('PUTDATA') same as file uploads for lower memory footprint postdata-CGI.pm-3.60.diff
Subject: postdata-CGI.pm-3.60.diff

Message body is not shown because it is too large.

Subject: Re: [rt.cpan.org #79102] PUTDATA/POSTDATA, uncorrupted by -utf8, and as tempfiles
Date: Mon, 20 Aug 2012 09:59:14 -0400
To: bug-CGI.pm [...] rt.cpan.org
From: Mark Stosberg <mark [...] summersault.com>
On 08/20/2012 08:53 AM, noreply via RT wrote: Show quoted text
> Mon Aug 20 08:53:18 2012: Request 79102 was acted upon. > Transaction: Ticket created by noreply > Queue: CGI.pm > Subject: PUTDATA/POSTDATA, uncorrupted by -utf8, and as tempfiles > Broken in: (no value) > Severity: Normal > Owner: Nobody > Requestors: > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=79102 > > > > inspired by http://stackoverflow.com/questions/11343663 > use CGI -utf8; would/could corrupt PUTDATA/POSTDATA -- fixed > I also add -putdata_upload to tread param('PUTDATA') same as file > uploads for lower memory footprint > > postdata-CGI.pm-3.60.diff
Thanks, We'll take a look. Mark
This issue has been copied to: https://github.com/leejo/CGI.pm/issues/100 please take all future correspondence there. This ticket will remain open but please do not reply here. This ticket will be closed when the github issue is dealt with.
Diff has been applied in a branch on github: https://github.com/leejo/CGI.pm/commit/66ab968abc427ba179da4ba6b93ebb5c8ea94835 Peer review would be appreciated. Not all the diff was applied (see commit message for reasoning).
commit 66ab968abc427ba179da4ba6b93ebb5c8ea94835 Author: Lee Johnson <lee@givengain.ch> Date: Sat Jul 12 16:32:40 2014 +0200 resolve #100 [rt.cpan.org #79102] - PUTDATA/POSTDATA now uncorrupted by -utf8, and done using tempfiles. the patch was sourced from https://rt.cpan.org/Public/Bug/Display.html?id=79102 and has been applied by hand as there were some changes in the diff that did not apply to this particular issue and that have already been fixed (#91) or rejected (#67)