Skip Menu |
 

This queue is for tickets about the JSON CPAN distribution.

Report information
The Basics
Id: 100423
Status: resolved
Priority: 0/
Queue: JSON

People
Owner: Nobody in particular
Requestors: avar [...] cpan.org
Cc:
AdminCc:

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



Subject: Please remove the experimental -convert_blessed_universally option
Download (untitled) / with headers
text/plain 583b
We hit an issue in production here at work where something stopped working because of the maxdepth problem described in RT #69620. This whole feature is just crazy IMO, nothing should be inserting stuff into UNIVERSAL at a distance like this, it means that loading random code that uses this -convert_blessed_universally option will screw with everything *globally*. We weren't even using JSON.pm in the code that broke, we were using JSON::XS. Please just remove this, it's not all that useful compared to just specifying options explicitly, and causes weird action at a distance.
Download (untitled) / with headers
text/plain 878b
On Tue Nov 18 20:06:01 2014, AVAR wrote: Show quoted text
> We hit an issue in production here at work where something stopped > working because of the maxdepth problem described in RT #69620. This > whole feature is just crazy IMO, nothing should be inserting stuff > into UNIVERSAL at a distance like this, it means that loading random > code that uses this -convert_blessed_universally option will screw > with everything *globally*. > > We weren't even using JSON.pm in the code that broke, we were using > JSON::XS. > > Please just remove this, it's not all that useful compared to just > specifying options explicitly, and causes weird action at a distance.
Sorry for the late reply. Unfortunately not a few modules used this option, but I hope this issue has been mitigated since JSON 2.91_01, at least for recent perls. If you still suffer from this issue, please reopen this. Thanks.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.