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

SearchableText "foo **" with a standalone wildcard causes error #362

Open
reebalazs opened this issue Jun 7, 2023 · 0 comments
Open

SearchableText "foo **" with a standalone wildcard causes error #362

reebalazs opened this issue Jun 7, 2023 · 0 comments
Assignees
Labels

Comments

@reebalazs
Copy link
Member

reebalazs commented Jun 7, 2023

Fix the problem and make "foo **" result in the same query as "foo". This currently causes an 500 Internal Server Error.

One might argue that "foo **" is not a valid query, to start with. However it can be expected as an input, thus it should not cause a cryptic error anyway. The fix should cause to yield the expected results by stripping out the standalone wildcard.

@reebalazs reebalazs self-assigned this Jun 7, 2023
@reebalazs reebalazs added the bug label Jun 7, 2023
reebalazs added a commit that referenced this issue Jun 7, 2023
Fix the problem and make "foo **" result in the same query as "foo".
Without the fix this used to cause an 500 Internal Server Error.

One might argue that "foo **"  is not a valid query, to start with.
However it can be expected as an input, thus it should not cause a
cryptic error anyway. The fix causes to yield the expected results by
stripping out the standalone wildcard.
reebalazs added a commit that referenced this issue Jun 7, 2023
Fix the problem and make "foo **" result in the same query as "foo".
Without the fix this used to cause an 500 Internal Server Error.

One might argue that "foo **"  is not a valid query, to start with.
However it can be expected as an input, thus it should not cause a
cryptic error anyway. The fix causes to yield the expected results by
stripping out the standalone wildcard.
tisto pushed a commit that referenced this issue Jun 9, 2023
Fix the problem and make "foo **" result in the same query as "foo".
Without the fix this used to cause an 500 Internal Server Error.

One might argue that "foo **"  is not a valid query, to start with.
However it can be expected as an input, thus it should not cause a
cryptic error anyway. The fix causes to yield the expected results by
stripping out the standalone wildcard.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant