-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Route params in /loki/api/v1/label/<name>/values
are tracked verbatim in request telemetry
#11549
Labels
good first issue
These are great first issues. If you are looking for a place to start, start here!
type/bug
Somehing is not working as expected
Comments
dannykopping
added
good first issue
These are great first issues. If you are looking for a place to start, start here!
type/bug
Somehing is not working as expected
labels
Dec 26, 2023
dannykopping
pushed a commit
to dannykopping/loki
that referenced
this issue
Dec 27, 2023
Fixes grafana#11549 Signed-off-by: Danny Kopping <danny.kopping@grafana.com>
8 tasks
dannykopping
pushed a commit
that referenced
this issue
Dec 27, 2023
**Which issue(s) this PR fixes**: Fixes #11549
dannykopping
pushed a commit
to dannykopping/loki
that referenced
this issue
Dec 27, 2023
…#11551) **Which issue(s) this PR fixes**: Fixes grafana#11549
dannykopping
pushed a commit
to dannykopping/loki
that referenced
this issue
Feb 19, 2024
…#11551) **Which issue(s) this PR fixes**: Fixes grafana#11549 (cherry picked from commit 18ea388)
dannykopping
pushed a commit
to dannykopping/loki
that referenced
this issue
Feb 20, 2024
…#11551) **Which issue(s) this PR fixes**: Fixes grafana#11549 (cherry picked from commit 18ea388)
dannykopping
pushed a commit
to dannykopping/loki
that referenced
this issue
Feb 26, 2024
…#11551) **Which issue(s) this PR fixes**: Fixes grafana#11549 (cherry picked from commit 18ea388)
rhnasc
pushed a commit
to inloco/loki
that referenced
this issue
Apr 12, 2024
…#11551) **Which issue(s) this PR fixes**: Fixes grafana#11549
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
good first issue
These are great first issues. If you are looking for a place to start, start here!
type/bug
Somehing is not working as expected
The
/loki/api/v1/label/<name>/values
endpoint accepts a path parameter, and this is used in theroute
label of theloki_request_duration_seconds_{bucket,sum,count}
metrics. This type of reflected user input is both dangerous and unnecessary as a label.Describe the bug
The given route input should not be reflected in the telemetry of the requests.
Expected behavior
The
/loki/api/v1/label/<name>/values
is rendered asloki_api_v1_label_values
without the user input.The text was updated successfully, but these errors were encountered: