Skip to content
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.

Sign up for GitHub

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

Jump to bottom

When Loki extracts label, it replaces \" with " and breaks the json #14166

Closed
doanbutar opened this issue Sep 18, 2024 · 0 comments · Fixed by #14499
Closed

When Loki extracts label, it replaces \" with " and breaks the json #14166

doanbutar opened this issue Sep 18, 2024 · 0 comments · Fixed by #14499
Assignees
@vlad-diachenko
Labels
area/logql component/logql type/bug Somehing is not working as expected

Comments

@doanbutar
Copy link
Contributor

doanbutar commented Sep 18, 2024 •
edited by vlad-diachenko
Loading

Describe the bug
When Loki extracts label params , it replaces " with " and breaks the json.

To Reproduce
Steps to reproduce the behavior:

  1. Go to this link

Image

Expected behavior
It should return valid JSON
@vlad-diachenko vlad-diachenko removed their assignment Sep 19, 2024
@vlad-diachenko vlad-diachenko self-assigned this Oct 14, 2024
@vlad-diachenko vlad-diachenko added component/logql type/bug Somehing is not working as expected area/logql labels Oct 14, 2024
vlad-diachenko added a commit that referenced this issue Oct 16, 2024
@vlad-diachenko
fix(logql): updated JSONExpressionParser to not unescape extracted va…
e711122 
…lues if it is JSON object.

re: #14166

Signed-off-by: Vladyslav Diachenko <[email protected]>
@vlad-diachenko vlad-diachenko mentioned this issue Oct 16, 2024
Merged
6 tasks
This was referenced Oct 16, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vlad-diachenko vlad-diachenko

Labels
area/logql component/logql type/bug Somehing is not working as expected
Projects
None yet
Milestone
No milestone
2 participants
@doanbutar @vlad-diachenko