waynr opened a new issue, #40097: URL: https://github.com/apache/arrow/issues/40097
### Describe the bug, including details regarding any error messages, version, and platform. I am working with an internal tool that establishes a flightsql gRPC connection to a thing. The tool sets [`DriverConfig.TLSEnabled`](https://github.com/apache/arrow/blob/a03d957b5b8d0425f9d5b6c98b6ee1efa56a1248/go/arrow/flight/flightsql/driver/config.go#L77) to `false` but I still see this error when the connection is being created: ``` grpc: the credentials require transport level security (use grpc.WithTransportCredentials() to set) ``` This error is coming from [grpc-go's `ClientConn.validateTransportCredential`](https://github.com/grpc/grpc-go/blob/3ae77e65289012f05699aef3567e68eed8ca9293/clientconn.go#L413) but only after calling [the `RequireTransportSecurity()` method of the `PerRPCCredentials` trait](https://github.com/grpc/grpc-go/blob/3ae77e65289012f05699aef3567e68eed8ca9293/credentials/credentials.go#L52). The [flightsql driver's `grpcCredentials`](https://github.com/apache/arrow/blob/a03d957b5b8d0425f9d5b6c98b6ee1efa56a1248/go/arrow/flight/flightsql/driver/utils.go#L55-L57) type implements this interface, but it doesn't seem to have any way to respect the aforementioned `DriverConfig.TLSEnabled` field. I am about to open a PR to propose threading the config parameter through and have flightsql's `grpcCredentials.RequireTransportSecurity()` implementation respect it (definitely open to alternative approaches). I am writing up this issue because the CONTRIBUTING doc suggested I do so before opening a PR. ### Component(s) FlightRPC, Go -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
