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

Mismatched quotes not raising exception in Classic API #254

Closed
JaimieMurdock opened this issue Jul 31, 2019 · 1 comment
Closed

Mismatched quotes not raising exception in Classic API #254

JaimieMurdock opened this issue Jul 31, 2019 · 1 comment
Labels
bug

Comments

@JaimieMurdock
Copy link
Contributor

via @DavidLFielding :

I expected mismatched quotes might cause havoc in the system or not return an error.

The query: http://127.0.0.1:5000/query?search_query=ti:"electron
with intentionally missing quote generates stack trace and exception:

"search.services.index.exceptions.IndexConnectionError: Problem communicating with ES: search_phase_execution_exception"

Again, these issues may be something to address down the road, but I just wanted to inform you of some of my experiences while trying to break the system.
@JaimieMurdock JaimieMurdock mentioned this issue Jul 31, 2019
Merged
JaimieMurdock added a commit that referenced this issue Aug 5, 2019
@JaimieMurdock
detecting mismatched quotes. close #254
083d652
@JaimieMurdock
Copy link
Contributor Author

closed in 083d652

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@JaimieMurdock