Closed Bug 917019 Opened 11 years ago Closed 11 years ago

Pass true_region and true_carrier from Fireplace to Zamboni

Categories

(Marketplace Graveyard :: API, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED
2013-10-01

People

(Reporter: cvan, Assigned: cvan)

References

Details

(Whiteboard: [qa-])

BEFORE:

With a valid, supported SIM:

1) Fireplace looks at the MCC/MNC pair from the SIM and passes the carrier and region to Zamboni.
2) Fireplace saves the carrier and region in localStorage.
3) User can change his/her region from https://marketplace.firefox.com/settings (carrier value remains unchanged until overridden by another change in the SIM's MCC/MNC value in the future)

Without a valid SIM:

1) Zamboni sends back in a header (API-Filter) of API response which region the user should be geolocated to.
2) Fireplace saves the carrier (null) and region in localStorage.
3) User can change his/her region from https://marketplace.firefox.com/settings


AFTER:

With a valid, supported SIM:

1) Fireplace looks at the MCC/MNC pair from the SIM and passes the carrier and region to Zamboni.
2) Fireplace saves the true_carrier and true_region in localStorage.
3) User can change his/her region from https://marketplace.firefox.com/settings but the true_carrier and true_region are the values we actually look at when determining whether to show a strictly restricted app.

With a valid, supported SIM:

1) Zamboni sends back in a header (API-Filter) of API response which region the user should be geolocated to.
2) Fireplace saves the true_carrier (null) and true_region in localStorage.
3) User can change his/her region from https://marketplace.firefox.com/settings but again no effect on strictly restricted apps.
Blocks: 896724
No longer blocks: content-control
The second heading in "AFTER:" should be "Without a valid SIM:"
Priority: -- → P3
https://github.com/mozilla/fireplace/commit/84a894b
Assignee: nobody → cvan
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-10-01
Please add STR here or mark it with [qa-] if no QA is needed.
Flags: needinfo?(cvan)
Flags: needinfo?(cvan)
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.