The default bug view has changed. See this FAQ.

JSONP request fails to wrap response when dots are present in the callback key

RESOLVED WONTFIX

Status

Webtools
BzAPI
RESOLVED WONTFIX
6 years ago
3 years ago

People

(Reporter: dbuc, Assigned: gerv)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
Please allow JSONP callback function keys to contain dots (.), the service currently fails to wrap the response at all when dots are present. There are many javascript libraries that use callback functions in this format - "JSONP.requests.map.0" - to internally separate different calls as they are returned.
(Assignee)

Comment 1

6 years ago
This is a bug in the Catalyst REST controller: Catalyst/Action/Serialize/JSONP.pm, which uses a "\w" regexp to test for validity, assuming that anything which doesn't match that is invalid.

I have never succeeded in getting my bugfixes applied to this module :-( I may have to fork yet more of it.

But in the mean time, I guess I could hack the copy on api-dev.

Gerv
(Reporter)

Comment 2

6 years ago
Do you have an idea of a ballpark ETA? I'm trying to balance the cost benefit of waiting vs developing my own request queue handler script. Best guess is fine :)
(Assignee)

Comment 3

6 years ago
Fixed for api-dev by hacking the module locally.

https://api-dev.bugzilla.mozilla.org/latest/bug?id=620789&content-type=application/javascript&callback=foo.bar

Gerv
(Reporter)

Comment 4

6 years ago
Thank you sooooooo much Gerv, this was a huge time saver!
(Assignee)

Comment 5

6 years ago
I'm going to call this FIXED. The upstream module isn't going to change <sigh>, and I'm hoping not to fork more of it. I've documented the limitation.

Gerv
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

6 years ago
Actually, no. I may have to fork this module after all.

Gerv
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 7

3 years ago
BzAPI is very much in maintenance mode and is being superseded by the BzAPI shim on top of the native Bugzilla REST API, and that native API itself. So this bug is not going to be fixed.

Gerv
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.