Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Brave not asking for permissions when loading geolocations #10864

Closed
kjozwiak opened this issue Sep 8, 2017 · 5 comments · Fixed by #10896
Closed

Brave not asking for permissions when loading geolocations #10864

kjozwiak opened this issue Sep 8, 2017 · 5 comments · Fixed by #10896

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Sep 8, 2017

Test plan

load https://browserleaks.com/geo within Brave


- Did you search for similar issues before submitting this one?

Yes

- Describe the issue you encountered:

Geolocation data is being loaded without prompting the user for permission.

- Platform (Win7, 8, 10? macOS? Linux distro?):

Win 7 x86 - Reproduced
Win 10 x64 - Reproduced
macOS 10.12.6 - Reproduced

- Brave Version (revision SHA):

Brave: 0.18.31
rev: a5a7280
Muon: 4.3.17
libchromiumcontent: 61.0.3163.71

- Steps to reproduce:

  1. load https://browserleaks.com/geo within Brave

You'll notice that the geolocation data will load without asking for the required permissions.

- Actual result:

Users geolocation data loads without asking the user for the required permissions.

- Expected result:

Brave should be prompting the user for permission before loading geolocation data.

- Will the steps above reproduce in a fresh profile? If not what other info can be added?

Yes

- Is this an issue in the currently released version?

No. Looks like it's a new regression in 0.18.31

- Can this issue be consistently reproduced?

Yes

- Screenshot if needed:

permissionissue

@kjozwiak
Copy link
Member Author

kjozwiak commented Sep 11, 2017

Still an issue with the latest preview build:

Brave: 0.18.32
rev: e94738d
Muon: 4.3.18
libchromiumcontent: 61.0.3163.71

Platforms checked:

  • Win 7 Pro x86 - Reproduced
  • Win 10 Pro x64 - Reproduced
  • macOS 10.12.6 x64 - Reproduced
  • Ubuntu 17.04 x64 - Reproduced

bridiver added a commit to brave/muon that referenced this issue Sep 11, 2017
bridiver added a commit that referenced this issue Sep 11, 2017
bridiver added a commit to brave/muon that referenced this issue Sep 11, 2017
NejcZdovc pushed a commit to NejcZdovc/browser-laptop that referenced this issue Sep 13, 2017
bbondy added a commit that referenced this issue Sep 13, 2017
This addresses review feedback for #10864
bbondy added a commit that referenced this issue Sep 13, 2017
This addresses review feedback for #10864
NejcZdovc pushed a commit to NejcZdovc/browser-laptop that referenced this issue Sep 13, 2017
bbondy added a commit to brave/muon that referenced this issue Sep 13, 2017
bbondy added a commit to brave/muon that referenced this issue Sep 13, 2017
@bbondy
Copy link
Member

bbondy commented Sep 13, 2017

This is all merged above for 4.3.21 and 0.18.x so marking this as complete.

@luixxiul
Copy link
Contributor

Was the fix merged to 0.19?

For me the geolocation issue on https://developer.mozilla.org/en-US/docs/Web/API/Geolocation/Using_geolocation is reproduced on:

Brave: 0.19.7
rev: c8481a9
Muon: 4.4.16
libchromiumcontent: 61.0.3163.71
V8: 6.1.534.31
Node.js: 7.9.0
Update Channel: Beta
OS Platform: Linux
OS Release: 4.9.0-3-amd64
OS Architecture: x64

@NejcZdovc
Copy link
Contributor

This was not merged into 0.19.x yet, see #10896

@bbondy
Copy link
Member

bbondy commented Sep 20, 2017

0.19.x: 16a6c86
0.20.x: 423706e
master: 5e5f089

bridiver added a commit that referenced this issue Sep 20, 2017
fix #10864
requires muon 4.3.19+
bridiver added a commit to brave/muon that referenced this issue Oct 4, 2017
bridiver pushed a commit to brave/muon that referenced this issue Oct 4, 2017
syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants