Skip to content
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

fix: Enable X509_V_FLAG_TRUSTED_FIRST flag in BoringSSL #31218

Conversation

@trop
Copy link
Contributor

@trop trop bot commented Sep 30, 2021

Backport of #31213

See that PR for details.

Notes: Fix Let's Encrypt DST Root CA X3 certificate expiration.

Fixes: #31212
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Copy link
Member

@VerteDinde VerteDinde left a comment

One extra patch

Loading

Copy link
Member

@clavin clavin left a comment

Extra patches

Loading

Copy link
Member

@MarshallOfSound MarshallOfSound left a comment

👍 bad merge fixed

Loading

clavin
clavin approved these changes Sep 30, 2021
Copy link
Member

@VerteDinde VerteDinde left a comment

Hold for linux-x64-testing

Loading

@VerteDinde VerteDinde merged commit d788d81 into 15-x-y Sep 30, 2021
10 of 14 checks passed
Loading
@VerteDinde VerteDinde deleted the trop/15-x-y-bp-fix-enable-x509_v_flag_trusted_first-flag-in-boringssl-1633033394829 branch Sep 30, 2021
@release-clerk
Copy link

@release-clerk release-clerk bot commented Sep 30, 2021

Release Notes Persisted

Fix Let's Encrypt DST Root CA X3 certificate expiration.

Loading

deepak1556 added a commit that referenced this issue Oct 1, 2021
* Revert "fix: Enable X509_V_FLAG_TRUSTED_FIRST flag in BoringSSL (#31218)"

This reverts commit d788d81.

* fix: remove expired DST Root CA X3
downace added a commit to vast-ru/electron-print-server that referenced this issue Oct 4, 2021
They fixed Let's Encrypt DST Root CA X3 certificate expiration for 12+ versions, see electron/electron#31218
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants