-
Notifications
You must be signed in to change notification settings - Fork 230
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
Stopped working with Brave #91
Comments
Here the sanitized HAR file: |
Brave unconditionally intercepts update requests to Google (even if not originating from the browser process...), and replaces it with a request to Someone reported this issue to me last year, but I unfortunately forgot to follow up to that. A work-around is to replace "clients2" with "clients3" since they seem to point to the same resources. Another workaround is to use two slashes in front of the path, since Brave's URL interception logic is based on a match pattern that is equivalent to I'll look into a way to detect Brave and add a work-around. |
The logic is applied to the CHROME build without further checks that specifically targets the Brave browser, because the server's behavior is the same either way. Moreover, I would more likely notice a behavioral regression in Chromium than Brave, because I do not use the latter myself.
It seems Brave has changed this code, so
|
- Rely on CORS instead of work-arounds for public XPI files from AMO (#92) - Add work-around for crx access in Brave (#91) - Support source viewing of extensions from Microsoft Edge (#95) - Remove web_accessible_resourced to avoid UUID leak (#100) - Correct 0-padding of hashes (#104) - Update jsbeautifier to v1.14.0 (#110) - Support source viewing of extensions from Thunderbird (#111) - Fix domain front of AMO in Firefox add-on (#115) - Recognize CRX3 files served by addons.opera.com (#118) - Opera only: Add work-around to access addons.opera.com (#119) - web only: Add crx keyword to input field for keyword search (#99) - web only: Avoid breakage of web version when an extension runs a script (#113) - Refactor: remove unsupported declarativeWebRequest path - Refactor: Use declarative page actions to show button.
Could you confirm whether version 1.6.12 fixes the issue for you? |
Cool thanks, I didn't install 1.6.12 but I installed 1.6.11 from Google official webstore and it's working... Not sure what happened, but anyway it's a good news :-) |
I'll remove the workaround with a future update then, as a cleanup. On Brave's end this probably started working again due to the changes from brave/brave-core@b3802c6 |
This reverts commit 791fef1. The work-around is not needed any more because Brave refactored, per https://github.com/Rob--W/crxviewer/issues/91#issuecomment-1080216994
When I try to look at a source:
Console:
Regards
The text was updated successfully, but these errors were encountered: