Skip to content

Bump transitive rustls-webpki dependency#165

Open
T0mstone wants to merge 1 commit into
typst:mainfrom
T0mstone:main
Open

Bump transitive rustls-webpki dependency#165
T0mstone wants to merge 1 commit into
typst:mainfrom
T0mstone:main

Conversation

@T0mstone
Copy link
Copy Markdown
Collaborator

...so that I can stop getting the annoying dependabot alert E-mails.
(I also considered changing my notif settings, but since there's apparently a vulnerability, why not stop people from running vulnerable code when running the tests that fetch something from the web)

We can then later remove the extra dependency line again after ureq updates its rustls dependency and we update to that newer ureq version.

@T0mstone T0mstone added the meta Discussion about the structure of this repo label May 12, 2026
@MDLC01
Copy link
Copy Markdown
Collaborator

MDLC01 commented May 12, 2026

I don't understand why we need rustls-webpki even though it's not used in the code

@T0mstone
Copy link
Copy Markdown
Collaborator Author

It's a transitive dependency (ureq depends on rustls, which depends on rustls-webpki). Adding an it as an explicit dependency tells Cargo's version resolver to use that version as the minimum, which is higher than what it would otherwise have chosen.

Comment thread Cargo.toml Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Discussion about the structure of this repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants