Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

metrics: resolve per-host inifinite cardinality #7306

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mohammed90 wants to merge 3 commits into master
base: master
Choose a base branch
Loading
from per-host-cardinality

Conversation

Copy link
Member

@mohammed90 mohammed90 commented Oct 15, 2025

Instructions to copilot are based on the discussion in #7208.

Assistance Disclosure

Copilot (Claude Sonnet 4) generated the code, and I verified it is correct.

Fixes #7208

Signed-off-by: Mohammed Al Sahaf <msaa1990@gmail.com>
@mohammed90 mohammed90 added bug 🐞 Something isn't working feature ⚙️ New feature or request labels Oct 15, 2025
Copy link
Member

@francislavoie francislavoie left a comment

Choose a reason for hiding this comment

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

Looks pretty good!

@francislavoie francislavoie added this to the v2.11.0 milestone Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@francislavoie francislavoie francislavoie approved these changes

Assignees

No one assigned

Labels

bug 🐞 Something isn't working feature ⚙️ New feature or request

Projects

None yet

Milestone

v2.11.0

Development

Successfully merging this pull request may close these issues.

High Cardinality Risk via Unsanitized Host Header in Caddy per_host Metrics

AltStyle によって変換されたページ (->オリジナル) /