Merge pull request #1541 from kokizzu/fix-clickhouse-proxy-issue

Fix clickhouse proxy issue, add support for database as query string DSN

Author: SpencerTorres

clickhouse_options.go

@@ -308,6 +308,8 @@ func (o *Options) fromDSN(in string) error {
 			o.Auth.Username = params.Get(v)
 		case "password":
 			o.Auth.Password = params.Get(v)
+		case "database":
+			o.Auth.Database = params.Get(v)
 		case "client_info_product":
 			chunks := strings.Split(params.Get(v), ",")
 

clickhouse_options_test.go

@@ -482,6 +482,21 @@ func TestParseDSN(t *testing.T) {
 			},
 			"",
 		},
+		{
+			"clickhouse proxy with database as query string",
+			"tcp://127.0.0.1/?database=bla",
+			&Options{
+				Protocol: Native,
+				TLS:      nil,
+				Addr:     []string{"127.0.0.1"},
+				Settings: Settings{},
+				Auth: Auth{
+					Database: `bla`,
+				},
+				scheme: "tcp",
+			},
+			"",
+		},
 	}
 
 	for _, testCase := range testCases {