Skip to content

add sanitized config endpoint#1003

Merged
jranson merged 7 commits into
trickstercache:mainfrom
jranson:jr/config-anon-endpoint
May 15, 2026
Merged

add sanitized config endpoint#1003
jranson merged 7 commits into
trickstercache:mainfrom
jranson:jr/config-anon-endpoint

Conversation

@jranson
Copy link
Copy Markdown
Member

@jranson jranson commented May 15, 2026

Description

This change adds a new sanitized config route on the listeners configured for the /config endpoint at /config/sanitized. The sanitized config provides a config output that is safe for sharing in Issues by anonymizing cache and backend names, as well as backend origin URLs, header injection values and authenticator data. See sanitize_test.go for exhaustive example input and output.

While implementing this capability, Codex found that 2 Options structs' Clone() functions were shallow-copying some elements. Since the running config is cloned and then the clone is mutated to generate the sanitized output, these options' Clones needed to be changed to a deep copy to ensure the sanitized config does not inadvertently mutate the running config.

Type of Change

    • Bug fix
    • New feature
    • Optimization
    • Test coverage
    • Documentation
    • Infrastructure

AI Disclosure

    • This contribution DOES NOT include AI-generated changes
    • This contribution DOES include AI-generated changes, and I have reviewed the relevant contributing guidelines.

jranson added 4 commits May 15, 2026 11:01
Signed-off-by: James Ranson <james@ranson.org>
Signed-off-by: James Ranson <james@ranson.org>
Signed-off-by: James Ranson <james@ranson.org>
Signed-off-by: James Ranson <james@ranson.org>
@jranson jranson requested a review from a team as a code owner May 15, 2026 17:11
jranson added 3 commits May 15, 2026 11:17
Signed-off-by: James Ranson <james@ranson.org>
Signed-off-by: James Ranson <james@ranson.org>
Signed-off-by: James Ranson <james@ranson.org>
@jranson jranson merged commit d9692d8 into trickstercache:main May 15, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants