Skip to content

Validate service names#83

Merged
bripkens merged 3 commits intomainfrom
validate-service-names
Apr 15, 2026
Merged

Validate service names#83
bripkens merged 3 commits intomainfrom
validate-service-names

Conversation

@mdonkers
Copy link
Copy Markdown
Member

Prevent certain names to be created by e.g. PEN testing

…ing tools might inject 'malicious' input into the attribute. This ends up being treated as a separate Website in Dash0.Have some sanitizing around the input, to strip names with characters that are unexpected.
Copy link
Copy Markdown
Member

@bripkens bripkens left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@bripkens bripkens merged commit 187e0cc into main Apr 15, 2026
15 checks passed
@bripkens bripkens deleted the validate-service-names branch April 15, 2026 11:36
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 15, 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.

2 participants