Don't ignore error in InitThriftClient (#16)

`thrift.NewTHttpClientWithOptions` may return an error, in which case `tTrans` is `nil`, which causes the next line to panic with the message `interface conversion: thrift.TTransport is nil, not *thrift.THttpClient` on the next line. Signed-off-by: Eric Bannatyne <[email protected]>
sigmacomputing · Dec 1, 2022 · 46f7675 · 46f7675
1 parent 5883857
commit 46f7675
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/internal/client/client.go b/internal/client/client.go
@@ -212,6 +212,10 @@ func InitThriftClient(cfg *config.Config) (*ThriftServiceClient, error) {
 			Timeout:   cfg.ClientTimeout,
 		}
 		tTrans, err = thrift.NewTHttpClientWithOptions(endpoint, thrift.THttpClientOptions{Client: httpclient})
+		if err != nil {
+			return nil, err
+		}
+
 		httpTransport := tTrans.(*thrift.THttpClient)
 		userAgent := fmt.Sprintf("%s/%s", cfg.DriverName, cfg.DriverVersion)
 		if cfg.UserAgentEntry != "" {