Loop server expects strict "application/json" content-type in requests while browsers (jquery?) send "application/json; charset=utf-8"

VERIFIED FIXED

Status

defect
P1
critical
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: NiKo, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

We should only check for "application/json" being part of the `Content-Type` header value.
Blocks: 979880
Severity: normal → critical
Priority: -- → P1
Assignee: nobody → nperriault
New attempt using req.accepts()
Attachment #8394930 - Attachment is obsolete: true
Attachment #8395202 - Flags: review?(ametaireau)
Thanks! https://github.com/mozilla-services/loop-server/commit/394ff4eda7f12ec26c0a8d1de73aeba4b95c5fe8
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Comment on attachment 8395202 [details] [review]
https://github.com/mozilla-services/loop-server/pull/43

clearing the needinfo. this landed already.
Attachment #8395202 - Flags: review?(ametaireau)
Fix either went out with a previous deployment to Production or has been superceded by new code...
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.