Skip to content

Loosen x509-parser bounds - #98

Merged
FlorianUekermann merged 1 commit into
FlorianUekermann:mainfrom
nmattia:patch-1
Jul 29, 2026
Merged

Loosen x509-parser bounds#98
FlorianUekermann merged 1 commit into
FlorianUekermann:mainfrom
nmattia:patch-1

Conversation

@nmattia

@nmattia nmattia commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

This changes the bounds on the x509-parser crate to allow using the latest version (0.18.1). The new bounds specify at least 0.16 (version in use currently) and up to but excluding whatever next breaking version x509-parser may release. In other words: allows 0.16.x, 0.17.x, 0.18.x; excludes 0.19.

This changes the bounds on the x509-parser crate to allow using the latest version (0.18.1). The new bounds specify _at least_ 0.16 (version in use currently) and _up to but excluding_ whatever next breaking version x509-parser may release. In other words: allows 0.16.x, 0.17.x, 0.18.x; excludes 0.19.
@nmattia

nmattia commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Note: I've verified that it builds fine with 0.17 & 0.18!

pull Bot pushed a commit to mikeyhodl/ic that referenced this pull request Jul 10, 2026
This bumps the version of x509-parser used directly by the codebase. We
still have an old 0.16 version pulled transitively via ic-bn-lib and
rustls-acme: FlorianUekermann/rustls-acme#98
@FlorianUekermann
FlorianUekermann merged commit c681c35 into FlorianUekermann:main Jul 29, 2026
37 checks passed
@nmattia
nmattia deleted the patch-1 branch July 30, 2026 09:19
nmattia added a commit to dfinity/ic that referenced this pull request Aug 11, 2026
This includes our changes to loosen the bounds which allows us to drop
some dependencies:

FlorianUekermann/rustls-acme#98
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