-
Notifications
You must be signed in to change notification settings - Fork 299
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
Fish 5978 Set TLS 1.3 as Default Protocol #5675
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM other than you need to update the copyright of the files you've edited - all but HttpConnectorAddress
need the portions copyright line updated to the current year
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strings.properties Needs copyright updating
Description
Payara6 changed to handle https requests using TLSv1.3 as default
Important Info
Blockers
None
Testing
New tests
None
Testing Performed
Testing Environment
Windows 10 (on Linux I think it will not work unless you have a certificate configured locally), JDK version "11.0.10" 2021-01-19 LTS, Maven 3.8.4"-->