Skip to content

fix: [OCISDEV-1115] harden nats communication - #12931

Merged
2403905 merged 3 commits into
masterfrom
backport/master/OCISDEV-1115
Sep 14, 2026
Merged

2403905 merged 3 commits into
masterfrom
backport/master/OCISDEV-1115

Conversation

@2403905

@2403905 2403905 commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Backport of #12927 to master.

Enhancement: Add TLS support for the frontend stat cache store connection

The frontend service was the only remaining place where a nats-js-kv store
connection could not be secured. Its OCS stat cache forwarded the store type,
nodes, database, table, TTL and credentials to reva, but not the TLS settings,
so the connection stayed plaintext with no operator toggle to change it. The
stat cache now honours OCIS_CACHE_ENABLE_TLS, OCIS_CACHE_TLS_INSECURE and
OCIS_CACHE_TLS_ROOT_CA_CERTIFICATE like every other cache and store.

@2403905
2403905 requested a review from a team as a code owner September 11, 2026 08:10
@kw-security

kw-security commented Sep 11, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

mmattel
mmattel previously approved these changes Sep 11, 2026
@2403905
2403905 enabled auto-merge (squash) September 11, 2026 08:15
2403905 added a commit that referenced this pull request Sep 11, 2026
@2403905
2403905 force-pushed the backport/master/OCISDEV-1115 branch from d72d09b to fadfb7b Compare September 11, 2026 08:52
@mzner
mzner self-requested a review September 14, 2026 07:22
@2403905
2403905 merged commit 0a97a84 into master Sep 14, 2026
77 checks passed
@2403905
2403905 deleted the backport/master/OCISDEV-1115 branch September 14, 2026 07:22
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.

4 participants