-
Notifications
You must be signed in to change notification settings - Fork 68
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
app/tls: Remove some options from SimpleTLSConfig.
Let go manage sane defaults, except MinVersion, a server can still use TLS 1.0 which seems a bit too old imo. - value from `curvePreferences` breaks go 1.19.6 so removed it. - `PreferServerCipherSuites` is deprecated. - Let go manage `CipherSuites` as it will take a safe default list.
- Loading branch information
Mathieu Borderé
committed
Mar 6, 2023
1 parent
2ef0aa7
commit 5fc49f0
Showing
2 changed files
with
13 additions
and
30 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters