-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
SecureSocket should use different certificate stores on different isolates #12491
Comments
Marked this as blocking #8227. |
Currently NSS does not support initialization more that once, and the database is process wide. There might be some work on changing this, see https://wiki.mozilla.org/NSS_Library_Init. Added this to the Later milestone. |
Removed Area-IO label. |
Removed this from the Later milestone. |
Removed Oldschool-Milestone-Later label. |
We are currently working on moving from using NSS to using BoringSSL, which have more options for handling different certificate collections. cc @whesse. |
The switch to BoringSSL has happened, and all certificates are set on a SecurityContext, which is local to a single isolate. This is in version 1.13, which is currenly on the dev channel, and will be on the stable channel in a couple of months. |
According to sgjesse's comment on issue #8227, SecureSocket shares a certificate store across all isolates. This violates the principle of isolation between isolates, as well as making issue #8227 unfixable. Each isolate should have its own certificate store that's independent of that in other isolates.
The text was updated successfully, but these errors were encountered: