Skip to content

Add skipTLSWarning config option#389

Open
ivanauth wants to merge 2 commits intoauthzed:mainfrom
ivanauth:fix/issue-180-disable-tls-warning
Open

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

Conversation

@ivanauth
Copy link

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 <[email protected]>
@ivanauth ivanauth force-pushed the fix/issue-180-disable-tls-warning branch from 0021c3a to 98b2812 Compare March 5, 2026 19:17
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.

Add capability to disable TLS warning

2 participants