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)

History Show all quoted text
  Tue Nov 18 06:06:01 2014 avar [...] cpan.org - Ticket created
Subject: Please remove the experimental -convert_blessed_universally option
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.
  Thu Dec 06 12:52:42 2018 ishigaki [...] cpan.org - Correspondence added
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.
  Thu Dec 06 12:52:43 2018 The RT System itself - Status changed from 'new' to 'open'
  Thu Dec 06 12:52:43 2018 ishigaki [...] cpan.org - Status changed from 'open' to 'resolved'