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.

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

fix invalid filter #682

Merged
merged 3 commits into from
Apr 24, 2020
Merged

fix invalid filter #682

merged 3 commits into from
Apr 24, 2020

Conversation

mason-fish
Copy link
Contributor

#626

In the repro steps for this issue, the log's contents have double quotes as part of the values which need to be escaped when we use that value as a search argument

escapeDoubleQuotesFilter

Mason Fish added 3 commits April 24, 2020 12:23
Signed-off-by: Mason Fish <mason@looky.cloud>
Signed-off-by: Mason Fish <mason@looky.cloud>
Signed-off-by: Mason Fish <mason@looky.cloud>
@mason-fish mason-fish requested review from a team and jameskerr April 24, 2020 22:32
Copy link
Member

@jameskerr jameskerr left a comment

Choose a reason for hiding this comment

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

Right on, looks like it works.

@mason-fish mason-fish merged commit 1fae48f into master Apr 24, 2020
@mason-fish mason-fish deleted the 626-fix_invalid_filter branch April 24, 2020 23:39
@philrz philrz linked an issue Apr 30, 2020 that may be closed by this pull request
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.

Query built by right-click "Filter" is invalid due to quotes on value
2 participants