Skip to content

Commit

Permalink
Revert 5d4c387 due to issue #70 on GitHub. Lock the min TLS version t…
Browse files Browse the repository at this point in the history
…o 1.2 instead
  • Loading branch information
NI committed Dec 23, 2021
1 parent c16f1f9 commit 366be28
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions application/server/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,15 +88,10 @@ func (s Server) Serve(
ssCfg := serverCfg.WithDefault()
l := s.logger.Context(
"Server (%s:%d)", ssCfg.ListenInterface, ssCfg.ListenPort)
cipherSuites := tls.CipherSuites() // only return secure ciphers
selectedCipherSuites := make([]uint16, 0, len(cipherSuites))
for _, s := range cipherSuites {
selectedCipherSuites = append(selectedCipherSuites, s.ID)
}
ss := &Serving{
server: http.Server{
Handler: handlerBuilder(commonCfg, ssCfg, l),
TLSConfig: &tls.Config{CipherSuites: selectedCipherSuites},
TLSConfig: &tls.Config{MinVersion: tls.VersionTLS12},
ReadTimeout: ssCfg.ReadTimeout,
ReadHeaderTimeout: ssCfg.InitialTimeout,
WriteTimeout: ssCfg.WriteTimeout,
Expand Down

0 comments on commit 366be28

Please sign in to comment.