Skip to content

Conversation

viktorking7
Copy link

Description

  • Removed insecure InsecureSkipVerify: true and set RootCAs to the provided CA pool to restore server certificate verification.
  • Kept mTLS settings intact (ClientCAs, ClientAuth: RequireAndVerifyClientCert) and enforced MinVersion: TLS1.2.
  • Improves security against MitM by validating the server’s cert chain.

Impact

  • More secure client connections; no interface changes required.
  • If hostname verification is desired, consider a follow-up to set tls.Config.ServerName via a dedicated ClientTLS(...).

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.

1 participant