Skip to content

Disallow unsupported TLSv1.3 socket options#3

Open
Maria-12648430 wants to merge 1 commit into
se-apc:issue/fix_for_tls1.3from
Maria-12648430:se-apc/fix_tlsv1.3_improved
Open

Disallow unsupported TLSv1.3 socket options#3
Maria-12648430 wants to merge 1 commit into
se-apc:issue/fix_for_tls1.3from
Maria-12648430:se-apc/fix_tlsv1.3_improved

Conversation

@Maria-12648430
Copy link
Copy Markdown

@adrigonzo I noticed your PR in ranch for this branch died down.
I guess there is interest in it, though, but I don't want to steal your PR. So... allow me to give you a hand here? =^^=
Regards, Maria 😊

@adrigonzo
Copy link
Copy Markdown

@Maria-12648430 thanks, I haven't had time to finish testing changes as per our discussion on ninenines#313. I was trying to implement the changes requested in the comments, but it's been quite busy.

Comment thread src/ranch_ssl.erl Outdated
end.

get_tls_versions_env() ->
case application:get_env(ssl, protocol_versions) of
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think is is called protocol_version in the env.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Indeed, fixed :)

@Maria-12648430 Maria-12648430 force-pushed the se-apc/fix_tlsv1.3_improved branch from 2c4621e to d2fb8d0 Compare June 2, 2021 11:30
@Maria-12648430
Copy link
Copy Markdown
Author

@Maria-12648430 thanks, I haven't had time to finish testing changes as per our discussion on ninenines#313. I was trying to implement the changes requested in the comments, but it's been quite busy.

Reminds me, tests will probably be required for this, also =^^=

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.

2 participants