Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

conf: redact scim.authToken in site config UI#62594

Merged
unknwon merged 1 commit into
mainfrom
jc/redact-scimauthtoken
May 10, 2024
Merged

conf: redact scim.authToken in site config UI#62594
unknwon merged 1 commit into
mainfrom
jc/redact-scimauthtoken

Conversation

@unknwon

@unknwon unknwon commented May 10, 2024

Copy link
Copy Markdown
Contributor

Test plan

CleanShot 2024-05-10 at 09 15 24@2x

@cla-bot cla-bot Bot added the cla-signed label May 10, 2024
@unknwon unknwon requested review from a team May 10, 2024 13:17
Comment thread internal/conf/validate.go
Comment on lines +207 to +209
// contains a "." character. For example,
// - "scim.authToken" => "scim\.authToken"
// - "dotcom": { "sams.clientSecret" } => "dotcom.sams\.clientSecret"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

love for doc string

@unknwon unknwon merged commit f9392f4 into main May 10, 2024
@unknwon unknwon deleted the jc/redact-scimauthtoken branch May 10, 2024 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants