Skip to content

Conversation

@barroco
Copy link
Contributor

@barroco barroco commented Oct 21, 2025

This PR enforces ssl and authentication of clients on ysql connections using client certificate.

@barroco barroco marked this pull request as ready for review October 22, 2025 16:12
@barroco barroco marked this pull request as draft October 22, 2025 16:13
@barroco barroco marked this pull request as ready for review October 22, 2025 16:13
@barroco barroco changed the title [security] Harness yugabyte cluster [security] Require client certificate to connect to yugabyte sql interface Oct 22, 2025
Copy link
Contributor

@mickmis mickmis left a comment

Choose a reason for hiding this comment

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

LGTM modulo comment.

Copy link
Contributor

@the-glu the-glu left a comment

Choose a reason for hiding this comment

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

Tested and confirmed working, however extrra flag can be removed :)


gflags:
master:
use_client_to_server_encryption: true
Copy link
Contributor

Choose a reason for hiding this comment

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

It's not needed to set this flag explicitly, it's handled in the state file via line 52 above (19d9d1ce3665e9d62d9a2538267116eb6a90644b9275b38ab30aeac127076243R52)

@barroco barroco force-pushed the harness-yugabyte-cluster branch from 5a26c74 to 2d6eb46 Compare January 2, 2026 14:13
@barroco barroco force-pushed the harness-yugabyte-cluster branch from 2d6eb46 to f1b6a51 Compare January 2, 2026 14:28
@barroco barroco merged commit 92be735 into interuss:master Jan 2, 2026
10 checks passed
@barroco barroco deleted the harness-yugabyte-cluster branch January 2, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants