-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Add publisher proof support for our CRX3 files #873
Comments
The code to generate proof is merged to Nightly. Now we need to support it in brave-core-crx-packager |
Now all brave CRXs released with a publisher proof. A PR to enforce publisher proof verification (brave/brave-core#12551) just have merged to brave-core. |
For QA: we need to check that all components are correctly installing on browser new profile. |
@brave/qa-team this originally caused a regression with Ads classification, so also confirm that all Ad components are installed correctly as well, version numbers are listed, no duplicates/errors, etc. |
@brave/qa-team could you also test that enabling this flag: And changing your setting to brave://settings/wallet to Crypto Wallets (deprecated) works when you load brave://wallet to load your old wallet? |
Verification
Verification PASSED on
|
Verification passed on
|
Case 5: Legacy Crypto Wallet - Geeta Completed
|
Inside this file:
chromium_src/components/component_updater/component_installer.cc
Landed in this PR:
https://github.com/brave/brave-core/pull/369/files
Revert from CRX3 back to using CRX3_WITH_PUBLISHER_PROOF once our CRX files include a publisher proof.... whatever that is :)
The text was updated successfully, but these errors were encountered: