Inn-app products can't be saved

VERIFIED FIXED in 2015-10-20

Status

P2
normal
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: madalin.cotetiu, Unassigned)

Tracking

Avenir
2015-10-20
Points:
---

Details

(Reporter)

Description

3 years ago
Steps to reproduce:
1. Open in app payments page for an app.
2. Click on Configure In-App Products and then on the Add a Product button.
3. Enter a name and a price point and then click the Save button.

Expected results:
The product is saved.

Actual results:
The request fails with a 404 error.

Notes/Issues:
Verified in FF44(Win7) Issue can be reproduced in mp-stage and mp-production.
Screencast for this issue: http://screencast.com/t/mWAJoX3C

Comment 1

3 years ago
I would wager that the 404 is something to do with the AWS move of marketplace. I remember there was something to do with an nginx setting that we had to flip?

Maybe bug 1066200?
Flags: needinfo?(kumar.mcmillan)

Updated

3 years ago
Iteration: --- → 44.3 - Nov 2
Or if jason can look into it.
Flags: needinfo?(jthomas)
Priority: -- → P2
Updated nginx config to disable merge_slashes https://github.com/mozilla-services/puppet-config/commit/507c5efed322bbab9a27a2bacbafc4f8722e92b4. Deployed to -dev and stage. Can we test again?
Flags: needinfo?(jthomas)
I think it's unlikely that it's an AWS issue. Here are my observations:

- Saving a product works fine for me in dev and stage using both Firefox 43 and 44.0a1 (2015-10-20) (the latest Nightly) on Mac
- the screencast showing the 404 URL makes it look like the `http` part is maybe not getting encoded. This URL is added to the DOM as a data attribute so perhaps decoding that value is failing. The full URL should look something like https://marketplace.allizom.org/api/v1/payments/http%3A%2F%2Fporcupine6139.testmanifest.com/in-app/

Perhaps it's a Windows only bug? Maybe it's fixed in the latest nightly? Madalin, could you copy and paste one of the exact URLs that you get a 404 on?
Flags: needinfo?(kumar.mcmillan)

Updated

3 years ago
Flags: needinfo?(madalin.cotetiu)
(Reporter)

Comment 5

3 years ago
I tested this in latest Nightly and I can't no longer reproduce this on stage but it still happens in production: http://screencast.com/t/5vAmsv9LyRT
This is the failing request: https://marketplace.firefox.com/api/v1/payments/http%3A%2F%2F86422260.testmanifest.com/in-app/

Maybe Jason's fix worked but it wasn't pushed to production as this is now working on stage? Screencast: http://screencast.com/t/dqFYVNrC
Flags: needinfo?(madalin.cotetiu)
The nginx changes have landed on prod. Can we test prod?
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Reporter)

Comment 7

3 years ago
Verified as fixed in FF44(Win7) in marketplace.firefox.com
The products can now be saved in production: http://screencast.com/t/j7igRyEJ1y
Closing bug.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.