Closed Bug 1253032 Opened 8 years ago Closed 8 years ago

Recent change to JSON::XS breaks some APIs

Categories

(bugzilla.mozilla.org :: API, defect)

Production
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dylan, Assigned: dylan)

Details

Attachments

(1 file)

https://metacpan.org/changes/distribution/JSON-XS#L9
"allow_nonref now affects booleans" which I think means it dies on them now.
Previously they were considered refs. We need to use the allow_nonref option to prevent this error.
Assignee: nobody → webservice
Component: General → WebService
Product: bugzilla.mozilla.org → Bugzilla
QA Contact: default-qa
Version: Production → unspecified
Which API methods are affected?
valid_login currently.
Right, bmo returns a bare "true" or "false" but upstream doesn't. :-/
Assignee: webservice → nobody
Component: WebService → API
Product: Bugzilla → bugzilla.mozilla.org
QA Contact: default-qa
Version: unspecified → Production
Attached patch 1253032_1.patchSplinter Review
Assignee: nobody → dylan
Attachment #8726186 - Flags: review?(dkl)
Comment on attachment 8726186 [details] [diff] [review]
1253032_1.patch

Review of attachment 8726186 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl
Attachment #8726186 - Flags: review?(dkl) → review+
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   9744f3c..97e70cc  master -> master
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: