Skip to content

test: fix TLS version test to support different SingleStore versions - #31

Merged
okramarenko merged 2 commits into
masterfrom
resolvetls
Nov 10, 2026
Merged

okramarenko merged 2 commits into
masterfrom
resolvetls

Conversation

@okramarenko

Copy link
Copy Markdown
Collaborator

No description provided.

@cursor

cursor Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
CI test config and SSL assertion logic only; no production connection or auth code paths change.

Overview
Updates SideBySide workflow configs to stop listing Tls13 under UnsupportedFeatures, so CI treats TLS 1.3 as available on the managed SingleStore target.

ConnectSslTlsVersion no longer derives the expected protocol from AppConfig.SupportedFeatures (including the old Linux TLS 1.3 upgrade path). It reads Ssl_version from the server, maps that string to SslProtocols, and asserts connection.SslProtocol matches in non-baseline builds; baseline only checks that a version was reported.

Reviewed by Cursor Bugbot for commit 76870bc. Configure here.

@okramarenko
okramarenko merged commit 8788ce6 into master Nov 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants