Skip to content

feat: add TLS URL parameters #2076

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat: add TLS URL parameters #2076

wants to merge 1 commit into from

Conversation

benweissmann
Copy link

Fixes #2024

@benweissmann benweissmann changed the title Add TLS URL parameters feat: Add TLS URL parameters Apr 23, 2022
@benweissmann benweissmann changed the title feat: Add TLS URL parameters feat: add TLS URL parameters Apr 25, 2022
vmihailenco
vmihailenco reviewed Jun 4, 2022
View reviewed changes
options.go

o.TLSConfig.MinVersion = uint16(q.int("TLSMinVersion"))
o.TLSConfig.MaxVersion = uint16(q.int("TLSMaxVersion"))
o.TLSConfig.InsecureSkipVerify = q.bool("TLSInsecureSkipVerify")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these options TLSMinVersion/TLSMaxVersion/TLSInsecureSkipVerify documented anywhere?

@vmihailenco
Copy link
Collaborator

Before merging this, we should check if these or similar options are already used anywhere. If not, we should consider using ?tls_min_version instead of ?TLSMinVersion.

@vmihailenco vmihailenco added the wait Can’t be processed temporarily for other reasons label Oct 6, 2022
@gaby gaby mentioned this pull request Sep 10, 2023
Closed
3 tasks
@gaby
Copy link

gaby commented Sep 10, 2023

@benweissmann @vmihailenco Any updates on this ?

@gaby gaby mentioned this pull request Sep 10, 2023
Open
@chayim chayim requested a review from ofekshenawa September 10, 2023 07:11
@apurvasaraiya
Copy link

@benweissmann @vmihailenco, any ETA for these changes to be released?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vmihailenco vmihailenco vmihailenco left review comments

@ofekshenawa ofekshenawa Awaiting requested review from ofekshenawa

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
wait Can’t be processed temporarily for other reasons
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support specifying TLS options in connection string
4 participants
@benweissmann @vmihailenco @gaby @apurvasaraiya