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.

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

[v10.4.x] Postgres: Allow disabling SNI on SSL-enabled connections #84249

Merged
merged 1 commit into from Mar 12, 2024

Conversation

grafana-delivery-bot[bot]
Copy link
Contributor

Backport 22d8258 from #83892


What is this feature?

Starting from version 1.10.7 lib/pq enables SNI.
However, this may be undesirable for some cases
Also, causes tls-mode verify-ca not to work correctly #65816

Why do we need this feature?

This introduces a new database configuration for disabling sslsni by setting it to 0.
By default is unset (fallbacks to the default lib/pq behaviour).

Who is this feature for?

[Add information on what kind of user the feature is for.]

Which issue(s) does this PR fix?:

Fixes #65816

Special notes for your reviewer:

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

* Postgres: Allow disabling SNI on SSL-enabled connections

* Update docs/sources/setup-grafana/configure-grafana/_index.md

Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com>

---------

Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com>
(cherry picked from commit 22d8258)
@papagian papagian merged commit 9bbd9cf into v10.4.x Mar 12, 2024
22 checks passed
@papagian papagian deleted the backport-83892-to-v10.4.x branch March 12, 2024 10:55
@mdvictor mdvictor modified the milestones: 10.4.x, 10.4.1 Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants