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: Use older version of x509-certificate for wasm32-unknown #934

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

cdmurph32
Copy link
Collaborator

Changes in this pull request

x509-certificate v0.22.0 uses a newer version of ring (0.17.5) which cannot be built by Xcode on Mac Silicon for wasm32-unknown-unknown.

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

Copy link

codecov bot commented Feb 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.94%. Comparing base (c6d58b8) to head (b7f14ec).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #934      +/-   ##
==========================================
+ Coverage   78.93%   78.94%   +0.01%     
==========================================
  Files         147      147              
  Lines       34842    34842              
==========================================
+ Hits        27503    27507       +4     
+ Misses       7339     7335       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

x509-certificate v0.22.0 uses a newer version of ring (0.17.5) which
cannot be built by Xcode on Mac Silicon for wasm32-unknown-unknown.
Copy link
Collaborator

@gpeacock gpeacock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simpler than I thought it would be, nice work!

@cdmurph32 cdmurph32 merged commit bd737c8 into main Feb 19, 2025
31 checks passed
@cdmurph32 cdmurph32 deleted the wasm_browser_x509 branch February 19, 2025 19:13
@scouten-adobe scouten-adobe mentioned this pull request Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants