Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Go][FlightSQL driver] Allow custom TLS configurations #35042

Closed
srebhan opened this issue Apr 11, 2023 · 0 comments · Fixed by #35051
Closed

[Go][FlightSQL driver] Allow custom TLS configurations #35042

srebhan opened this issue Apr 11, 2023 · 0 comments · Fixed by #35051
Assignees
@srebhan
Labels
Component: FlightRPC Component: Go Type: enhancement
Milestone
12.0.0

Comments

@srebhan
Copy link
Contributor

srebhan commented Apr 11, 2023 •
edited
Loading

Describe the enhancement requested

Currently, the database/sql FlightSQL driver does not allow for TLS connections. However, it should be possible to enable TLS connections and also specify custom TLS parameters such as certificates to check against, minimum version etc.

This was also discussed in the initial driver PR #34331.

Component(s)

FlightRPC, Go
@srebhan srebhan changed the title Allow custom TLS configurations [Go][FlightSQL driver] Allow custom TLS configurations Apr 11, 2023
@srebhan srebhan mentioned this issue Apr 11, 2023
Merged
@github-actions github-actions bot mentioned this issue Apr 11, 2023
Merged
zeroshade pushed a commit that referenced this issue Apr 12, 2023
@srebhan @disq
GH-35042: [Go][FlightSQL driver] Add TLS configuration (#35051)
631c68a 
This PR adds the ability to enable and customize TLS parameters for Golang's `database/sql` driver.
* Closes: #35042

Lead-authored-by: Sven Rebhan <srebhan@influxdata.com>
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
Co-authored-by: Kemal <223029+disq@users.noreply.github.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
@zeroshade zeroshade added this to the 12.0.0 milestone Apr 12, 2023
liujiacheng777 pushed a commit to LoongArch-Python/arrow that referenced this issue May 11, 2023
@srebhan @disq @liujiacheng777
apacheGH-35042: [Go][FlightSQL driver] Add TLS configuration (apache#…
dc55f22 
…35051)

This PR adds the ability to enable and customize TLS parameters for Golang's `database/sql` driver.
* Closes: apache#35042

Lead-authored-by: Sven Rebhan <srebhan@influxdata.com>
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
Co-authored-by: Kemal <223029+disq@users.noreply.github.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
ArgusLi pushed a commit to Bit-Quill/arrow that referenced this issue May 15, 2023
@srebhan @disq @ArgusLi
apacheGH-35042: [Go][FlightSQL driver] Add TLS configuration (apache#…
b591263 
…35051)

This PR adds the ability to enable and customize TLS parameters for Golang's `database/sql` driver.
* Closes: apache#35042

Lead-authored-by: Sven Rebhan <srebhan@influxdata.com>
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
Co-authored-by: Kemal <223029+disq@users.noreply.github.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
rtpsw pushed a commit to rtpsw/arrow that referenced this issue May 16, 2023
@srebhan @disq @rtpsw
apacheGH-35042: [Go][FlightSQL driver] Add TLS configuration (apache#…
a3d5218 
…35051)

This PR adds the ability to enable and customize TLS parameters for Golang's `database/sql` driver.
* Closes: apache#35042

Lead-authored-by: Sven Rebhan <srebhan@influxdata.com>
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
Co-authored-by: Kemal <223029+disq@users.noreply.github.com>
Signed-off-by: Matt Topol <zotthewizard@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@srebhan srebhan

Labels
Component: FlightRPC Component: Go Type: enhancement
Projects
None yet
Milestone
12.0.0
Development

Successfully merging a pull request may close this issue.

GH-35042: [Go][FlightSQL driver] Add TLS configuration srebhan/arrow
2 participants
@zeroshade @srebhan