Skip to content

Conversation

skeggse
Copy link
Contributor

@skeggse skeggse commented Jun 24, 2024

What does this PR do?

Adds a feature flag to control the TLS implementation used internally by the client to fix #157.

Additional Notes

Review checklist

Please check relevant items below:

  • This PR includes all newly recorded cassettes for any modified tests. (N/A)
  • This PR does not rely on API client schema changes.
    • The CI should be fully passing.
  • Or, this PR relies on API schema changes and this is a Draft PR to include tests for that new functionality.
    • Note: CI shouldn't be run on this Draft PR, as its expected to fail without the corresponding schema changes.

@skeggse skeggse requested a review from a team as a code owner June 24, 2024 19:49
@nkzou nkzou changed the title feat: support rustls feature flag Support rustls with feature flag via reqwest Jun 26, 2024
@nkzou nkzou merged commit d40075d into DataDog:master Jun 26, 2024
@nkzou
Copy link
Collaborator

nkzou commented Jun 26, 2024

Hi there, thanks for your contribution - we're planning to cut a new release next week, so it should be updated on crates.io by then.

@skeggse
Copy link
Contributor Author

skeggse commented Jun 26, 2024

Thank you!

@skeggse skeggse deleted the rustls branch June 26, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support rustls-tls-webpki-roots feature for reqwest
2 participants