Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Google Map embed not render #7280

Closed
chinkung opened this issue Dec 9, 2019 · 3 comments
Closed

Google Map embed not render #7280

chinkung opened this issue Dec 9, 2019 · 3 comments
Labels
google needs-investigation A bug not 100% confirmed/fixed OS/Android Fixes related to Android browser functionality OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/No release-notes/exclude webcompat/not-shields-related Sites are breaking because of something other than Shields. workaround/allow-all-cookies

Comments

@chinkung
Copy link

chinkung commented Dec 9, 2019

Description

Brave has a problem with website embed Google Map content with API restricted by HTTP referrers

Steps to Reproduce

  1. Use Brave browser to navigate to https://www.zaolong.in.th/branch/st-louis/

Actual result:

The error is show up on embed map area

Google Maps Platform rejected your request. This IP, site or mobile application is not authorized to use this API key. Request received from IP address 184.22.230.174, with referer: https://www.google.com/

Expected result:

Static Map is rendered on the browser

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 1.0.1 Chromium: 78.0.3904.108 (Official Build) (32-bit)
Revision 4b26898a39ee037623a72fcfb77279fce0e7d648-refs/branch-heads/3904@{#889}
OS Windows 10 OS Version 1909 (Build 18363.476)
JavaScript V8 7.8.279.23
Flash (Disabled)
User Agent Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.108 Safari/537.36
Command Line "C:\Program Files\BraveSoftware\Brave-Browser\Application\brave.exe" --profile-directory=Default --enable-dom-distiller --disable-domain-reliability --no-pings --extension-content-verification=enforce_strict --extensions-install-verification=enforce --enable-oop-rasterization=Enabled --sync-url=https://no-thanks.invalid --enable-features=WebUIDarkMode,NewExtensionUpdaterService,SimplifyHttpsIndicator,PasswordImport --disable-features=AutofillServerCommunication,UnifiedConsent,AudioServiceOutOfProcess,SyncUSSBookmarks,NotificationTriggers,SmsReceiver,LookalikeUrlNavigationSuggestionsUI --flag-switches-begin --flag-switches-end
Executable Path C:\Program Files\BraveSoftware\Brave-Browser\Application\brave.exe

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the dev channel?
  • Can you reproduce this issue with the nightly channel?

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
    Yes
  • Is the issue reproducible on the latest version of Chrome?
    No

Miscellaneous Information:

  1. This issue can be mitigated on client side by disable Brave Shield
  2. This issue can be mitigated on server side by add https://www.google.com/* to Google Map API's HTTP referrers list
@bsclifton bsclifton added feature/geolocation webcompat/not-shields-related Sites are breaking because of something other than Shields. needs-investigation A bug not 100% confirmed/fixed and removed feature/geolocation labels Dec 9, 2019
@ghost
Copy link

ghost commented Dec 25, 2019

To further drill down, you can resolve the issue by allowing all cookies

@rebron
Copy link
Collaborator

rebron commented Feb 7, 2020

I think we need a new Social Media blocking pref for this or fold into "Allow Google logins".
@ryanbr Do you have suggestions for rules for this one?

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Feb 7, 2020
@rebron rebron added the google label Mar 10, 2020
@pes10k
Copy link
Contributor

pes10k commented Jul 1, 2020

fixed with #8696

@pes10k pes10k closed this as completed Jul 1, 2020
@pes10k pes10k added this to the 1.12.x - Beta milestone Jul 24, 2020
@bbondy bbondy added OS/Android Fixes related to Android browser functionality OS/Desktop labels Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
google needs-investigation A bug not 100% confirmed/fixed OS/Android Fixes related to Android browser functionality OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/No release-notes/exclude webcompat/not-shields-related Sites are breaking because of something other than Shields. workaround/allow-all-cookies
Projects
None yet
Development

No branches or pull requests

5 participants