mirror of
https://github.com/benbusby/whoogle-search.git
synced 2026-04-25 12:15:50 +03:00
[GH-ISSUE #1050] [BUG] 500 Internal Server Error #642
Labels
No labels
Fixed (Pending PR Merge)
Stale
bug
enhancement
enhancement
good first issue
help wanted
keep-open
needs more info
pull-request
question
theme
unfortunate
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/whoogle-search#642
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @seirdotexe on GitHub (Aug 10, 2023).
Original GitHub issue: https://github.com/benbusby/whoogle-search/issues/1050
Describe the bug
When you search for Mathias Normann, you get error code 500. It doesn't happen when you search for Mathias Normannaaa.
To Reproduce
Steps to reproduce the behavior:
Deployment Method
runexecutableVersion of Whoogle Search
Desktop (please complete the following information):
Additional context
@ahmad-alkadri commented on GitHub (Aug 13, 2023):
Confirmed getting the same bug on my docker instance.
It seems that the problem's coming from the parsing of the query. Curious if there's other instances of search queries producing this bug.
EDIT: Found the problem. This function:
github.com/benbusby/whoogle-search@18fc14dc5b/app/filter.py (L35-L47)will return the first element of an array in
['q']key of the result of theparse_qsfunction. The problem was there could be an instance where the value would be blank. And theparse_qsis from theurllibmodule, which is a built-in python module.Luckily according to the doc of the function we could keep the blank value instead of throwing it. I'll try it and try to make a PR asap if successful.