Skip to content

💥 Enable TLS if api key specified#567

Merged
THardy98 merged 8 commits intomainfrom
enable-tls-api-key
Dec 9, 2025
Merged

💥 Enable TLS if api key specified#567
THardy98 merged 8 commits intomainfrom
enable-tls-api-key

Conversation

@THardy98
Copy link
Contributor

@THardy98 THardy98 commented Dec 2, 2025

What was changed

DWISOTT

  1. Part of Assume TLS enabled if API key provided in SDKs features#687

  2. How was this tested:
    Unit tests

  3. Any docs updates needed?
    Maybe

@THardy98 THardy98 changed the title enable TLS if api key specified 💥 Enable TLS if api key specified Dec 2, 2025
@THardy98 THardy98 force-pushed the enable-tls-api-key branch 2 times, most recently from c5c054f to b3277e8 Compare December 2, 2025 17:37
@THardy98 THardy98 marked this pull request as ready for review December 2, 2025 19:41
@THardy98 THardy98 requested a review from a team as a code owner December 2, 2025 19:41
@THardy98 THardy98 requested a review from cretz December 3, 2025 19:14
Copy link
Member

@cretz cretz left a comment

Choose a reason for hiding this comment

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

But would also like @jmaeagle99's approval

@THardy98 THardy98 requested a review from jmaeagle99 December 4, 2025 17:57
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.

3 participants