Conversation
70d104e to
7315894
Compare
yezhizhen
left a comment
There was a problem hiding this comment.
This is really nice feature.
|
Does that change prevent embedders to use the webpki roots on platforms other than Android? |
|
Currently, as I understand it, you can still override the certificates as an embedder. |
How? I could only find the |
|
Oh I thought that is what you meant. With this PR there is the option to use a custom store via "certificate_path". Otherwise it will use the certificate system from the platform. If it doesn't have any way to get certificates from the platform it will fall back to webpki roots. Can you explain the reason why an embedder would want the webpki roots specifically instead of relying on the platform which gives better security guarantees? |
Because the platform does not necessarily give better security guarantees, and you may not want to be tied to the OS policy on that front. Firefox by default ships with its own roots, and can be setup to also lookup OS provided ones (useful for enterprise deployments). It would be better to have an explicit toggle (eg. a pref) to control that behavior. |
|
Ok I added a way to just use the webpki verifier. |
Thanks!
That looks reasonable to me. |
|
🔨 Triggering try run (#19896088516) for Android, OpenHarmony |
|
|
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
5b2d0f8 to
4f3a21f
Compare
Signed-off-by: Narfinger <[email protected]>
|
🔨 Triggering try run (#19907392915) for Android, OpenHarmony |
|
✨ Try run (#19907392915) succeeded. |
This changes the certificate verifier to use rustls_platform_verifier under the hood.
Note: The rustls-platform-verifier has specific instructions for android and currently we fall back to the webpki roots for this platform.
Testing: This was tested on linux, macos and ohos and the rest in the CI.
Fixes: Should fix #32903 and partially #35227