-
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
[Security][Android] SSL Pinning doesn't work #24453
Comments
Upstream appears to be preparing to roll this out to Android: https://chromium.googlesource.com/chromium/src/+/f12eac9342eb88971851d46e31c97af8c2c27%5E%21/ |
This is now working in Brave for Android and so we should enable too. This was done on desktop in brave/brave-core#8750. |
It's probably just a matter of removing the check for Android here: https://github.com/brave/brave-core/blob/44a98b1a20d935db5eab9899361871740695d29b/chromium_src/net/http/transport_security_state.cc#L17-L25 and then testing it like this:
|
Verified on
STEPS:
ACTUAL RESULTS:
|
Description
SSL Pinning doesn't work for the domains listed in https://github.com/brave/brave-core/blob/master/chromium_src/net/tools/transport_security_state_generator/input_file_parsers.cc
Steps to reproduce
Actual result
Webpage loads without an SSL error
Expected result
Webpage fails to load with an SSL error
Issue reproduces how often
Version/Channel Information:
Device details
Brave version
Website problems only
Additional information
The text was updated successfully, but these errors were encountered: