If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Loop production should return the call url directly, rather than doing a 302.

VERIFIED FIXED

Status

Hello (Loop)
Server
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: alexis, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa?])

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Comment 1

3 years ago
This is a configuration change, and I think Benson already did it. Benson, can you confirm and add a pointer to the config change here?
Flags: needinfo?(bwong)
(Reporter)

Comment 2

3 years ago
OH, this is actually some code changes that need to happen on the server, and not only a configuration value to change.
Flags: needinfo?(bwong)
Whiteboard: [qa?]
(Reporter)

Comment 3

3 years ago
Created attachment 8438552 [details] [review]
link to github PR
Attachment #8438552 - Flags: review?(rhubscher)
Attachment #8438552 - Flags: feedback?(standard8)
Comment on attachment 8438552 [details] [review]
link to github PR

https://github.com/mozilla-services/loop-server/commit/408bd150fe9900209f4c5a4e14a9941e1daad886
Attachment #8438552 - Flags: review?(rhubscher) → review+
https://github.com/mozilla-services/loop-server/commit/408bd150fe9900209f4c5a4e14a9941e1daad886
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Comment on attachment 8438552 [details] [review]
link to github PR

Seems reasonable to me.
Attachment #8438552 - Flags: feedback?(standard8) → feedback+
Verified in code and through unit tests.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.