Skip to content

[tls] Allow TLS certificate and key to come from bytes#504

Merged
smklein merged 10 commits intomainfrom
bytes_tls
Dec 14, 2022
Merged

[tls] Allow TLS certificate and key to come from bytes#504
smklein merged 10 commits intomainfrom
bytes_tls

Conversation

@smklein
Copy link
Copy Markdown
Contributor

@smklein smklein commented Dec 7, 2022

ConfigTls is now an enum, and can either by supplied AsFile or AsBytes.

Fixes #490

@smklein smklein changed the title Allow TLS certificate and key to come from bytes [tls] Allow TLS certificate and key to come from bytes Dec 7, 2022
@smklein smklein marked this pull request as ready for review December 7, 2022 03:28
@smklein smklein requested review from ahl and jclulow December 7, 2022 03:29
Copy link
Copy Markdown
Collaborator

@ahl ahl left a comment

Choose a reason for hiding this comment

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

looks good; this is a breaking change, yes?

pinging @davepacheco in case he wants to take a look

@davepacheco
Copy link
Copy Markdown
Collaborator

Thanks for the ping. Don't wait for me!

@smklein
Copy link
Copy Markdown
Contributor Author

smklein commented Dec 14, 2022

looks good; this is a breaking change, yes?

Updated in 0ebb21e

Base automatically changed from refresh_tls to main December 14, 2022 20:23
@smklein smklein enabled auto-merge (squash) December 14, 2022 20:24
@smklein smklein merged commit 9e12b70 into main Dec 14, 2022
@smklein smklein deleted the bytes_tls branch December 14, 2022 20: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