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.

Sign up for GitHub

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

Jump to bottom

fix(ext/node): rewrite X509Certificate resource and add `publicKey()` #24988

Merged

littledivy merged 2 commits into denoland:main from littledivy:x509_yoke

Aug 11, 2024

Member

littledivy commented Aug 11, 2024 •

edited

Loading

Changes:

Remove unsafe usage, rewrite Rust representation with yoke.
Implement X509Certificate.prototype.publicKey()

Fixes #23307

littledivy added 2 commits

August 11, 2024 13:32


          fix(ext/node): rewrite X509Certificate

b719c9e


          Rewrite

littledivy requested a review from lucacasonato

August 11, 2024 09:33

lucacasonato approved these changes

View reviewed changes

littledivy merged commit b61fd62 into denoland:main

17 checks passed

littledivy deleted the x509_yoke branch

August 11, 2024 13:28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet