Skip to content

enable native-tls to verify cerver CA & patch user/pwd login responce parsing#153

Open
alexshagiev wants to merge 3 commits into
jmgilman:masterfrom
alexshagiev:master
Open

enable native-tls to verify cerver CA & patch user/pwd login responce parsing#153
alexshagiev wants to merge 3 commits into
jmgilman:masterfrom
alexshagiev:master

Conversation

@alexshagiev
Copy link
Copy Markdown

it appears the feature for native-tls already present in the cargo.toml but the code does not allow enabling this feature. I understand that using Mutual TLS is still not possible but at least this way it is possible to use OS cert store to use reqwest's native-tls

@alexshagiev alexshagiev changed the title enable native-tls to verify cerver CA enable native-tls to verify cerver CA & patch user/pwd login responce parsing Oct 14, 2025
@alexshagiev
Copy link
Copy Markdown
Author

@jmgilman , what do you think can this be merged & released to crates.io?

@stormshield-gt
Copy link
Copy Markdown
Collaborator

Thanks for submitting the PR !

For the same reasons described https://github.com/seanmonstar/reqwest/issues/2025issuecomment-2913873836 I'm thinking of ditching the native-tls feature. Now, we are sure it's not used, as you've shown it's currently broken.

Now, we use reqwest with the webpki feature, which doesn't load the OS cert store. If instead we used rustls-platform-verifier, which also uses the OS cert store, would it satisfy your needs ?

Note: rustls-platform-verifier is not natively supported by reqwest yet but there is a workaround

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