Skip to content

Add skipTLSWarning config option#389

Merged
tstirrat15 merged 2 commits intoauthzed:mainfrom
ivanauth:fix/issue-180-disable-tls-warning
Apr 28, 2026
Merged

Add skipTLSWarning config option#389
tstirrat15 merged 2 commits intoauthzed:mainfrom
ivanauth:fix/issue-180-disable-tls-warning

Conversation

@ivanauth
Copy link
Copy Markdown
Contributor

Summary

Add a new config option skipTLSWarning that allows users to suppress the TLS warning in the Status.Conditions when running SpiceDB without TLS in internal/development environments.

Fixes #180

Changes

  • Add skipTLSWarning config key (boolean, defaults to false)
  • Add SkipTLSWarning field to SpiceConfig struct
  • Update TLS warning logic to check the flag
  • Add test case for the new functionality

Usage

apiVersion: authzed.com/v1alpha1
kind: SpiceDBCluster
spec:
  config:
    datastoreEngine: memory
    skipTLSWarning: true  # Suppress TLS warning for internal deployments

Testing

go test ./pkg/config/... -v -run "TestNewConfig/memory_with_skipTLSWarning"

ecordell
ecordell previously approved these changes Mar 5, 2026
@ecordell ecordell added this pull request to the merge queue Mar 5, 2026
github-merge-queue Bot pushed a commit that referenced this pull request Mar 5, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 5, 2026
ivanauth added 2 commits March 5, 2026 14:15
Add a new config option 'skipTLSWarning' that allows users to suppress
the TLS warning in the Status.Conditions when running SpiceDB without
TLS in internal/development environments.

Signed-off-by: ivanauth <ivan@authzed.com>
@ivanauth ivanauth force-pushed the fix/issue-180-disable-tls-warning branch from 0021c3a to 98b2812 Compare March 5, 2026 19:17
@tstirrat15 tstirrat15 added this pull request to the merge queue Apr 28, 2026
Merged via the queue into authzed:main with commit 680cffd Apr 28, 2026
11 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add capability to disable TLS warning

3 participants