Closed Bug 1094430 Opened 10 years ago Closed 10 years ago

Cannot login to FxA on -dev using 1.1 (or FF18)

Categories

(Marketplace Graveyard :: Consumer Pages, defect)

Avenir
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: scolville, Unassigned)

References

Details

STR: 

  Login with FF18 on -dev.

What happens:

  Login window is dismissed but the login fails to complete

What should happen.

  Login completes successfully and the page is updated to show you are signed in.

The initial bug to fix why the login doesn't work in FF18 and FFOS 1.1 was fixed here 1093747

As we were working-around the lack of window.location.origin pre Gecko21 elsewhere changes were made to move to using a polyfill.

This caused this bug due to polyfill not being required in settings_local. 

You can see this by running require('settings'); in the console.

the Fix for this has been landed but the code doesn't appear to be deployed on -dev. Looking at  https://marketplace-dev-cdn.allizom.org/media/fireplace/js/include.js?b=1415213857181 there are only 2 strings matching 'polyfill' where I'm expecting 3. 

See also:

define("settings_local",[],function(){var e=window.location.origin;return{api_url:e,media_url:document.body.getAttribute("data-media"),iframe_installer_src:e+"/iframe-install.html",iframe_potatolytics_src:e+"/potatolytics.html",potatolytics_enabled:!1,tracking_enabled:!0}})

That first array should contain 'polyfill'.

The patch was landed here: 

https://github.com/mozilla/fireplace/commit/453fff28efe999a6438cd8eb5086f6b16fb161e2

Once the deployment issue is resolved this should work.
Blocks: 1093747
No longer blocks: 1093747
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Tested this after a subsequent rev was deployed and it's working now.
Blocks: 1007956
Additional QA notes - you can use FFOS 1.1 or FF18 on -dev to test this.
You need to log in before you can comment on or make changes to this bug.