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

Web Store Update URL Missing, causing source-attribution confusion #19069

Closed
jonathansampson opened this issue Oct 28, 2021 · 6 comments · Fixed by brave/brave-core#10781
Closed

Comments

@jonathansampson
Copy link
Contributor

Description

Extensions installed from the Chrome Web Store are claiming to be not from the web store. This includes extensions like Google Translate and more, which clearly are from the Chrome Web Store.

Steps to Reproduce

  1. Install Google Translate
  2. Navigate to brave://extensions/
  3. Note state of extension

Actual result:

The extension is not properly identified as having originated from the Web Store.

image

Reproduces how often:

Always.

Brave version (brave://version info)

1.32.84, 1.32.85, 1.33.37

Version/Channel Information:

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

Other Additional Information:

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

Miscellaneous Information:

https://github.com/brave/brave-core/blob/9929ff6411a14350dd61fd19a37e93726b44e64d/common/extensions/brave_extensions_client.cc#L47 assumes a kComponentUpdater switch, never falling back on extension_urls::GetDefaultWebstoreUpdateUrl() in its absence.

@stephendonner
Copy link

stephendonner commented Nov 3, 2021

Verified PASSED using

Brave 1.33.58 Chromium: 95.0.4638.69 (Official Build) nightly (x86_64)
Revision 6a1600ed572fedecd573b6c2b90a22fe6392a410-refs/branch-heads/4638@{#984}
OS macOS Version 11.6.1 (Build 20G224)

Steps:

  1. new profile
  2. launched Brave
  3. installed Google Translate
  4. opened brave://extensions
  5. examined the state of the extension

Confirmed I no longer saw the Not from Web Store. messaging.

default developer mode
Screen Shot 2021-11-03 at 3 10 43 PM Screen Shot 2021-11-03 at 3 10 46 PM

Verified PASSED using

Brave 1.33.82 Chromium: 96.0.4664.45 (Official Build) beta (64-bit)
Revision 76e4c1bb2ab4671b8beba3444e61c0f17584b2fc-refs/branch-heads/4664@{#947}
OS Windows 10 Version 20H2 (Build 19042.1348)

Steps:

  1. new profile
  2. launched Brave
  3. installed Google Translate
  4. opened brave://extensions
  5. examined the state of the extension

Confirmed I no longer saw the Not from Web Store. messaging.

default developer mode
19069-1 19069-2

@JimBreton
Copy link

FYI it looks like this bug made it into Release 1.32.106.

@jonathansampson
Copy link
Contributor Author

@JimBreton Apologies; we're working on getting this uplifted and delivered.

@rebron rebron added release/blocking priority/P1 A very extremely bad problem. We might push a hotfix for it. priority/P2 A bad problem. We might uplift this to the next planned release. labels Nov 17, 2021
@stephendonner
Copy link

FYI it looks like this bug made it into Release 1.32.106.

thanks for the heads-up; this should be in the next release of 1.32.x (as soon as we can cut it, with a few other crucial fixes).

@kjozwiak
Copy link
Member

Clearing the QA Pass labels. We should re-check this once we get an RC.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 23, 2021

Verification passed on

Brave | 1.32.111 Chromium: 96.0.4664.45 (Official Build) (64-bit)
-- | --
Revision | 76e4c1bb2ab4671b8beba3444e61c0f17584b2fc-refs/branch-heads/4664@{#947}
OS | Windows 10 Version 21H1 (Build 19043.1348)

  • Reproduced the issue in 1.32.106
  • Verified the STR from the description and ensured Not from Web Store messaging is not shown in brave://extensions/ in 1.32.111
1.32.106 1.32.111
image image

Verified passed with

Brave | 1.32.111 Chromium: 96.0.4664.45 (Official Build) (x86_64)
-- | --
Revision | 76e4c1bb2ab4671b8beba3444e61c0f17584b2fc-refs/branch-heads/4664@{#947}
OS | macOS Version 11.6.1 (Build 20G224)
  • Reproduced the issue in 1.32.106
  • Verified the STR from the description and ensured Not from Web Store messaging is not shown in brave://extensions/ in 1.32.111
  • Verified in both clean and upgrade profiles
1.32.106 1.32.111
106 111

Verification passed on

Brave 1.32.112 Chromium: 96.0.4664.45 (Official Build) (64-bit)
Revision 76e4c1bb2ab4671b8beba3444e61c0f17584b2fc-refs/branch-heads/4664@{#947}
OS Ubuntu 18.04 LTS
image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment