Skip to content

[feat] Update config sanitizer (tracing & cache endpoints)#1004

Merged
jranson merged 2 commits into
trickstercache:mainfrom
jranson:jr/config-sanitize-tracing
May 17, 2026
Merged

[feat] Update config sanitizer (tracing & cache endpoints)#1004
jranson merged 2 commits into
trickstercache:mainfrom
jranson:jr/config-sanitize-tracing

Conversation

@jranson
Copy link
Copy Markdown
Member

@jranson jranson commented May 16, 2026

Description

This updates the /config/sanitized endpoint to sanitize redis cache and tracing endpoints, and skips sanitization for known-safe caching headers like Cache-Control and Expires.

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 2 commits May 15, 2026 17:48
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 16, 2026 00:00
Copy link
Copy Markdown
Contributor

@LimitlessEarth LimitlessEarth left a comment

Choose a reason for hiding this comment

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

lgtm

@jranson jranson merged commit 1607b36 into trickstercache:main May 17, 2026
8 checks passed
crandles pushed a commit to crandles/trickster that referenced this pull request May 27, 2026
…cache#1004)

* sanitize tracing names

* sanitize redis endpoints; don't sanitize caching headers

---------

Signed-off-by: James Ranson <james@ranson.org>
Signed-off-by: Chris Randles <randles.chris@gmail.com>
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