Skip to content

Commit

Permalink
enable tls
Browse files Browse the repository at this point in the history
  • Loading branch information
serprex committed Feb 12, 2024
1 parent 9ffcb66 commit 9cab0b6
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions flow/connectors/clickhouse/clickhouse.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ package connclickhouse

import (
"context"
"crypto/tls"
"database/sql"
"fmt"

Expand Down Expand Up @@ -85,6 +86,9 @@ func connect(ctx context.Context, config *protos.ClickhouseConfig) (*sql.DB, err
Username: config.User,
Password: config.Password,
},
TLS: &tls.Config{
InsecureSkipVerify: true,

Check failure on line 90 in flow/connectors/clickhouse/clickhouse.go

View workflow job for this annotation

GitHub Actions / lint

G402: TLS InsecureSkipVerify set true. (gosec)

Check failure

Code scanning / CodeQL

Disabled TLS certificate check High

InsecureSkipVerify should not be used in production code.
},
})

if err := conn.PingContext(ctx); err != nil {
Expand Down

0 comments on commit 9cab0b6

Please sign in to comment.