geolocation not working in Firefox 51 release

VERIFIED FIXED in Firefox 51

Status

()

Core
Geolocation
--
major
VERIFIED FIXED
4 months ago
a month ago

People

(Reporter: asa, Assigned: glandium)

Tracking

({regression})

51 Branch
mozilla54
x86_64
Windows 10
regression
Points:
---
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox51blocking verified, firefox52+ verified, firefox53 verified, firefox54 verified)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

4 months ago
Geolocation doesn't work at Google Maps or any other site I've tested on Windows Firefox 51 release build.

Steps:

1. visit https://www.google.com/maps
2. click the location button on the map
3. say yes to the confirmation dialog

Results

Several seconds pass and then "Your location could not be determined."

Also tested at https://www.mozilla.org/en-US/firefox/geolocation/ and at http://www.w3schools.com/HTML/tryit.asp?filename=tryhtml5_geolocation_error and both fail as well.

Tested on Mac and everything seems to work.
API keys are missing in Windows builds (tested 32 bit build). From nsURLFormatter.js in the build:
>    MOZILLA_API_KEY:  () => "no-mozilla-api-key",
>    GOOGLE_API_KEY:   () => "no-google-api-key",
>    BING_API_CLIENTID:() => "no-bing-api-clientid",
>    BING_API_KEY:     () => "no-bing-api-key",
This appears to be a regression in 51. I can repro the problem in 51, but not in 50.1. I can't reproduce in Aurora 52 or Nightly 54, but those pre-release channels use Mozilla's location service instead of Microsoft's.
status-firefox51: --- → affected
status-firefox52: --- → ?
status-firefox53: --- → unaffected
status-firefox54: --- → unaffected
tracking-firefox51: --- → +
tracking-firefox52: --- → +
Last good: 50.0b11
First bad: 51.0b1

jlund, are you aware of changes in releng which could cause this?
Flags: needinfo?(jlund)
Is this also affecting Firefox for Android?
Kan-Ru/Josh/Garvan, can any of you investigate? I can reproduce on my laptop if you'd like to try locally, Josh (or Garvan - I'll buy you a beer ;).
Flags: needinfo?(kchen)
Flags: needinfo?(josh)
Flags: needinfo?(garvankeeley+bmo)

Comment 6

4 months ago
(In reply to Sebastian Hengst [:aryx][:archaeopteryx] (needinfo on intermittent or backout) from comment #3)
> Last good: 50.0b11
> First bad: 51.0b1
> 
> jlund, are you aware of changes in releng which could cause this?

I'm not aware of anything releng related that would affect this. Not ruling it out, just saying it is unlikely. Releng focuses on the engineering of 'how to ship code', rather than the management of 'what code is shipped'.

release-mgmt + devs would have a better idea of what exactly landed on 51.0b1; likely something that got migrated from mozilla-aurora->mozilla-beta. I would bet this affected aurora/developer-edition when 51 got migrated from central/nightly.
Flags: needinfo?(jlund)
jcristau suggests https://bugzilla.mozilla.org/show_bug.cgi?id=1294585 as potential cause.
Don't have 51 release yet, but tried 51 Beta and noticed the same issue that Asa described for Google maps. However, once I moved onto the firefox geolocation test page, it worked right away, went back to Google maps and it ended up finding my location there as well.
^ this was meant for Android, sorry.
Looks like it's windows specific. Tried on linux64 and win7 (64bit), the latter didn't work.

Mark, do you happen to know if we changed anything related to https://dxr.mozilla.org/build-central/source/relabs-puppet/modules/slave_secrets/manifests/mozilla_geoloc_api_keys.pp for windows recently?

The release log https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-win64/1484771846/mozilla-release-win64-bm73-build1-build38.txt.gz passes the path as:

12:50:40     INFO -    --with-mozilla-api-keyfile=/c/builds/mozilla-desktop-geoloc-api.key
Flags: needinfo?(mcornmesser)
jcristau just mentioned bug 1294585 in IRC, can be related
(In reply to Rail Aliiev [:rail] ⌚️ET from comment #10)
> Looks like it's windows specific. Tried on linux64 and win7 (64bit), the
> latter didn't work.
> 
> Mark, do you happen to know if we changed anything related to
> https://dxr.mozilla.org/build-central/source/relabs-puppet/modules/
> slave_secrets/manifests/mozilla_geoloc_api_keys.pp for windows recently?
> 
> The release log
> https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-
> win64/1484771846/mozilla-release-win64-bm73-build1-build38.txt.gz passes the
> path as:
> 
> 12:50:40     INFO -   
> --with-mozilla-api-keyfile=/c/builds/mozilla-desktop-geoloc-api.key

I am not seeing any changes made to it. I just verified that they were on the 2008 instances and have what appears to be usable permissions.
Flags: needinfo?(mcornmesser)
Adding Mike per comment 7.
Flags: needinfo?(mh+mozilla)

Comment 14

4 months ago
> Kan-Ru/Josh/Garvan, can any of you investigate? I can reproduce on my laptop
> if you'd like to try locally, Josh (or Garvan - I'll buy you a beer ;).

If it helps, I can take a look in a few hours. I just got a Windows 10 gaming box, I *suppose* I can use it for other things. (In reply to Andrew Overholt [:overholt] from comment #5).
Thanks, Garvan. It looks like it's something that gets substituted at build time on the build machines so not likely something you can help with. That beer is still on offer :)
Flags: needinfo?(kchen)
Flags: needinfo?(josh)
Flags: needinfo?(garvankeeley+bmo)
I don't think this is related but I am the "owner" of the GLS account (especially since Doug left us) :)
(fwiw, it's likely that safebrowsing is also broken due to the missing keys)
From https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-win64/1484771846/mozilla-release-win64-bm73-build1-build38.txt.gz:

12:50:41     INFO -  checking for zip... c:/mozilla-build/info-zip/zip.EXE
12:50:41     INFO -  checking for the Mozilla API key... no
12:50:41     INFO -  checking for the Google API key... no
12:50:41     INFO -  checking for the Bing API key... no
12:50:41     INFO -  checking for the Adjust SDK key... no

from https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-linux/1484771846/mozilla-release-linux-bm77-build1-build45.txt.gz:

12:52:32     INFO -  checking for zip... /usr/bin/zip
12:52:32     INFO -  checking for the Mozilla API key... yes
12:52:32     INFO -  checking for the Google API key... yes
12:52:32     INFO -  checking for the Bing API key... no
12:52:32     INFO -  checking for the Adjust SDK key... no

These are coming from https://dxr.mozilla.org/mozilla-central/source/toolkit/moz.configure#551-563 

Summoning gps, :) he may have some insights on how those calls are converted into filenames.
Flags: needinfo?(gps)

Comment 19

4 months ago
running mozregression with geo.wifi.uri set to "https://www.googleapis.com/geolocation/v1/geolocate?key=%GOOGLE_API_KEY%" indeed confirms this will be due to bug 1294585
regression-range https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=52be5024e34261bcd08d68fda65379ed0ffd363d&tochange=8c3529c5f60a1b292eaa981b07c27091040a04cc
Blocks: 1294585
status-firefox52: ? → affected
status-firefox53: unaffected → disabled
status-firefox54: unaffected → disabled
Keywords: regressionwindow-wanted

Comment 20

4 months ago
This is a release blocker.
tracking-firefox51: + → blocking
(In reply to Rail Aliiev [:rail] ⌚️ET from comment #18)
> 12:50:41     INFO -  checking for the Mozilla API key... no
> 12:50:41     INFO -  checking for the Google API key... no

Can we update the configure to fail the build when the beta or release branding are being used but cannot find api keys?
(Assignee)

Comment 22

4 months ago
(In reply to Sylvestre Ledru [:sylvestre] from comment #21)
> (In reply to Rail Aliiev [:rail] ⌚️ET from comment #18)
> > 12:50:41     INFO -  checking for the Mozilla API key... no
> > 12:50:41     INFO -  checking for the Google API key... no
> 
> Can we update the configure to fail the build when the beta or release
> branding are being used but cannot find api keys?

That's bug 1256730 and is waiting for bug 1293643 to be fixed... but now I realize maybe the actual problem is what's causing *this* bug, being that the path is an msys path and configure expects a windows path.
Flags: needinfo?(mh+mozilla)
(Assignee)

Updated

4 months ago
Assignee: nobody → mh+mozilla

Comment 23

4 months ago
glandium provided an answer. I'll be around for the next few hours to review any changes. Just ping me on IRC to get my attention.
Flags: needinfo?(gps)
Thanks!
Comment hidden (mozreview-request)

Comment 26

4 months ago
mozreview-review
Comment on attachment 8830052 [details]
Bug 1333516 - Fix windows path for API keys after bug 1294585.

https://reviewboard.mozilla.org/r/106976/#review108028
Attachment #8830052 - Flags: review?(gps) → review+

Comment 27

4 months ago
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/56147a136a8c
Fix windows path for API keys after bug 1294585. r=gps
(Assignee)

Comment 28

4 months ago
I have a try build with this patch + bug 1256730 running, and I also want to see what happens on that autoland push before pushing to other branches.
(Assignee)

Comment 29

4 months ago
The TC builds are good. Waiting for buildbot builds. As for bug 1256730, it looks like only cross-osx builds are a problem at the moment.

Comment 30

4 months ago
Hi GPS, please nominate patches for uplift to Aurora53, Beta52 and Release51 when ready. Thanks! We are hoping to gtb 51.0.1 in the next couple of hours, mainly driven by this issue.
Flags: needinfo?(gps)

Comment 31

4 months ago
Redirecting needinfo to glandium since the changes are his and he is actively testing things.
Flags: needinfo?(gps) → needinfo?(mh+mozilla)
(Assignee)

Comment 32

4 months ago
Comment on attachment 8830052 [details]
Bug 1333516 - Fix windows path for API keys after bug 1294585.

We're good to go, the buildbot builds are good.

Approval Request Comment
[Feature/Bug causing the regression]: regression from bug 1294585
[User impact if declined]: Geolocation doesn't work
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: verified on autoland
[Needs manual test from QE? If yes, steps to reproduce]: see comment 0
[List of other uplifts needed for the feature/fix]: we may want to push bug 1256730 along when it's ready to land (which is not the case currently)
[Is the change risky?]: no
[Why is the change risky/not risky?]: The patch is changing a non-working path to a working path in a mozconfig.
[String changes made/needed]: none
Flags: needinfo?(mh+mozilla)
Attachment #8830052 - Flags: approval-mozilla-release?
Attachment #8830052 - Flags: approval-mozilla-beta?
Attachment #8830052 - Flags: approval-mozilla-aurora?
(Assignee)

Updated

4 months ago
Duplicate of this bug: 1293643
Liz wanted this pushed to release ASAP, so I went ahead and landed it on m-r before the official a+:
https://hg.mozilla.org/releases/mozilla-release/rev/cac8c9e7e0efdb615504f66ede2cffeddca348ee
I'll wait for official approval before landing on the other branches
status-firefox51: affected → fixed

Comment 35

4 months ago
Comment on attachment 8830052 [details]
Bug 1333516 - Fix windows path for API keys after bug 1294585.

We need this landed on 51+
Attachment #8830052 - Flags: approval-mozilla-release?
Attachment #8830052 - Flags: approval-mozilla-release+
Attachment #8830052 - Flags: approval-mozilla-beta?
Attachment #8830052 - Flags: approval-mozilla-beta+
Attachment #8830052 - Flags: approval-mozilla-aurora?
Attachment #8830052 - Flags: approval-mozilla-aurora+

Comment 36

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/56147a136a8c
Status: NEW → RESOLVED
Last Resolved: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54

Comment 37

4 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/9cb32c41b8cb
status-firefox52: affected → fixed
Flags: qe-verify+
status-firefox53: disabled → affected
status-firefox54: disabled → fixed

Comment 38

4 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/7d4459bdab3d
status-firefox53: affected → fixed
https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-release-win64/1485304068/mozilla-release-win64-bm94-build1-build32.txt.gz looks much better:

16:53:14     INFO -  checking for zip... c:/mozilla-build/info-zip/zip.EXE
16:53:14     INFO -  checking for the Mozilla API key... yes
16:53:14     INFO -  checking for the Google API key... yes
16:53:14     INFO -  checking for the Bing API key... no
16:53:14     INFO -  checking for the Adjust SDK key... no
The issue is verified fixed on 51.0.1-build2 (20170124203848) using Windows 10 x64, Windows 7 x64, Mac OS X 10.11 and Ubuntu 16.04 x64.
status-firefox51: fixed → verified
See Also: → bug 1333783
I reported bug 1333783 to make sure that we have automation to avoid this situation in the future.
Discussing with Andrei, he confirmed that we tested safe browsing in beta7:
https://wiki.mozilla.org/Releases/Firefox_51/Test_Plan#Beta_7
https://drive.google.com/file/d/0B6_25UfoAoJhUHgtMkVHaVlMQVU/view
and it was working.
The phishing test pages work because toolkit adds them to its database: https://dxr.mozilla.org/mozilla-central/rev/c989c7b352279925edf138373e4ca3f1540dbd5f/toolkit/components/url-classifier/SafeBrowsing.jsm#330
See Also: → bug 1336920
Tested on :
Nightly 54.0a1 (2017-02-09)
Aurora 53.0a2  (2017-02-09)
Beta 52.0b4    (20170206101855)

Verified fixed on Windows 10 , Ubuntu 16.04 and Mac OS X 10.11 , using the links from comment 0: https://www.google.com/maps
https://www.mozilla.org/en-US/firefox/geolocation/ 
http://www.w3schools.com/HTML/tryit.asp?filename=tryhtml5_geolocation_error.
Status: RESOLVED → VERIFIED
status-firefox52: fixed → verified
status-firefox53: fixed → verified
status-firefox54: fixed → verified
You need to log in before you can comment on or make changes to this bug.