Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

example(bindings): client hello cb example #4385

Merged
merged 16 commits into from
Mar 27, 2024

Conversation

jmayclin
Copy link
Contributor

@jmayclin jmayclin commented Jan 27, 2024

This commit adds an example showing how to use the client hello callback to set different configs on connections.

Description of changes:

This adds a very basic example server that configures different config's based on the SNI's in the client hello.

Call-outs:

I added this under a separate "examples" folder rather than adding it in the s2n-tls-tokio examples folder. This is the same way that s2n-quic manages their examples, which I appreciate, because it makes the example dependencies much clearer.

Testing:

This PR also adds a github workflow to test the example.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This commit adds an example showing how to use the client hello callback
to set different configs on connections.
@github-actions github-actions bot added the s2n-core team label Jan 27, 2024
@jmayclin jmayclin marked this pull request as ready for review January 29, 2024 17:32
@lrstewart lrstewart self-requested a review January 30, 2024 11:25
@jmayclin jmayclin requested a review from camshaft February 5, 2024 19:10
@jmayclin jmayclin requested a review from lrstewart March 26, 2024 21:48
@jmayclin jmayclin enabled auto-merge (squash) March 26, 2024 22:06
@jmayclin jmayclin merged commit ba825b8 into aws:main Mar 27, 2024
32 checks passed
@jmayclin jmayclin deleted the client-hello-config-resolution branch July 1, 2024 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants