New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch bug #100926, compatibility with CGI 4 (with fallback). #9

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@thrig

thrig commented Apr 3, 2015

Nabbed code from https://rt.cpan.org/Public/Bug/Display.html?id=100926 to avoid warnings with CGI 4. List context use looks safe here, as this code is assigning to an array, and not the security hazardous form, unless the "object_param_cache" item is somehow used unsafely, elsewhere. Tests pass w/o warnings.

@markstos

This comment has been minimized.

Show comment
Hide comment
@markstos

markstos Apr 3, 2015

Owner

Looks good to me. Merging. Could you leave a comment in RT that this has been merged?

Thanks.

Owner

markstos commented Apr 3, 2015

Looks good to me. Merging. Could you leave a comment in RT that this has been merged?

Thanks.

@markstos

This comment has been minimized.

Show comment
Hide comment
@markstos

markstos Apr 3, 2015

Owner

At thumbnail size, I thought your Gravatar was a photo of the top of a bloody, cracked skull. I was relieved to see when I clicked through your profile that it is fresh-baked bread instead.

Owner

markstos commented Apr 3, 2015

At thumbnail size, I thought your Gravatar was a photo of the top of a bloody, cracked skull. I was relieved to see when I clicked through your profile that it is fresh-baked bread instead.

@gquipster

This comment has been minimized.

Show comment
Hide comment
@gquipster

gquipster Oct 7, 2016

Hi Mark, can you let me know when this will be merged and published to CPAN? Thanks Gareth

gquipster commented Oct 7, 2016

Hi Mark, can you let me know when this will be merged and published to CPAN? Thanks Gareth

@markstos

This comment has been minimized.

Show comment
Hide comment
@markstos

markstos Oct 7, 2016

Owner

@gquipster the module is currently in need of a new maintainer. The original author was @tjmather, but neither he nor I are active Perl developers now.

Owner

markstos commented Oct 7, 2016

@gquipster the module is currently in need of a new maintainer. The original author was @tjmather, but neither he nor I are active Perl developers now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment