Skip to content

fix: Respect system certificates#289

Merged
figsoda merged 1 commit intonix-community:masterfrom
AiyionPrime:fix/use-system-certificates
Feb 20, 2026
Merged

fix: Respect system certificates#289
figsoda merged 1 commit intonix-community:masterfrom
AiyionPrime:fix/use-system-certificates

Conversation

@AiyionPrime
Copy link
Contributor

in order to use nix-index behind proxies, which impose a certain certificate on the traffic.

this resolves #288

in order to use nix-index behind proxies, which impose a certain certificate on the traffic.
@AiyionPrime
Copy link
Contributor Author

@RaitoBezarius or @bennofs might one of you take a look at the suggestion in the PR in the next weeks?

@AiyionPrime
Copy link
Contributor Author

@RaitoBezarius is there something I could do to get this reviewed?

@AiyionPrime
Copy link
Contributor Author

Or @bennofs or @figsoda for that matter?

Copy link
Member

@figsoda figsoda left a comment

Choose a reason for hiding this comment

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

looks like this feature is removed in 0.13, what is the alternative in 0.13?

@figsoda figsoda merged commit 059454b into nix-community:master Feb 20, 2026
5 checks passed
@AiyionPrime AiyionPrime deleted the fix/use-system-certificates branch February 20, 2026 21:19
@AiyionPrime
Copy link
Contributor Author

The last sentence in the commit message that introduced the change gives a hint:

This greatly simplifies the configuration matrix of what could be enabled, but
also means the removal of some parts that some people may have been using. The
consolidated API should still allow users to do these things.

For example, instead of adding webpki-roots with a feature flag, you should use
the webpki-roots crate yourself, and pass them into tls_certs_only(certs).

Originally posted by @seanmonstar in seanmonstar/reqwest@a59cdeb

So I assume we could do something similar with rustls-tls-native-roots and rustls-native-certs; and maybe even rustls-platform-verifier is applicable.

But I haven't used either, yet.

@AiyionPrime
Copy link
Contributor Author

... a conclusion I apparently reached at last year, too :D

This might be a favorable solution in order to stick with rustls instead of the system native resolver.

https://github.com/rustls/rustls-platform-verifier

edit implemented least invasively in #289.

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.

[bug] additional certificates and proxies

2 participants