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

Adds dtls 1.3 support in TLS::Proxy #23375

Closed

Conversation

fwh-dc
Copy link
Contributor

@fwh-dc fwh-dc commented Jan 23, 2024

Dependent on #23319

@t8m t8m added branch: master Merge to master branch triaged: feature The issue/pr requests/adds a feature labels Jan 23, 2024
@t8m t8m added this to the DTLS-1.3 milestone Jan 23, 2024
@fwh-dc fwh-dc changed the base branch from master to feature/dtls-1.3 March 21, 2024 13:08
@fwh-dc fwh-dc force-pushed the dtls13-support-in-tls-proxy branch from cbaa311 to 8c942b8 Compare March 21, 2024 13:11
@fwh-dc
Copy link
Contributor Author

fwh-dc commented Mar 21, 2024

@mattcaswell this PR is a dependency for other PR's so it should be prioritized when you get to look at DTLS 1.3 PR's

@fwh-dc fwh-dc force-pushed the dtls13-support-in-tls-proxy branch from 8c942b8 to cd9dce7 Compare March 21, 2024 13:53
@fwh-dc
Copy link
Contributor Author

fwh-dc commented Mar 21, 2024

@t8m This is ready for review. Would you please add labels to reflect that? Thanks!

@t8m t8m added approval: review pending This pull request needs review by a committer approval: otc review pending This pull request needs review by an OTC member tests: present The PR has suitable tests present and removed branch: master Merge to master branch labels Apr 4, 2024
@t8m t8m removed the approval: otc review pending This pull request needs review by an OTC member label Apr 4, 2024
@mattcaswell mattcaswell added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Apr 5, 2024
@openssl-machine openssl-machine added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Apr 6, 2024
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@t8m
Copy link
Member

t8m commented Apr 8, 2024

Merged to the feature branch. Thank you for your contribution.

@t8m t8m closed this Apr 8, 2024
openssl-machine pushed a commit that referenced this pull request Apr 8, 2024
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #23375)
openssl-machine pushed a commit that referenced this pull request Apr 22, 2024
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #23375)
mattcaswell pushed a commit to mattcaswell/openssl that referenced this pull request Apr 22, 2024
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from openssl#23375)
nhorman pushed a commit to nhorman/openssl that referenced this pull request Apr 22, 2024
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from openssl#23375)
openssl-machine pushed a commit that referenced this pull request Apr 23, 2024
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #23375)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: ready to merge The 24 hour grace period has passed, ready to merge tests: present The PR has suitable tests present triaged: feature The issue/pr requests/adds a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants