bad request returned on certain searches #112
Labels
No labels
area: app c
area: app d
area: devops
area: discovery
area: docs
area: proposal
area: X-device Sync
Chainquery
consider soon
dependencies
Epic
Fix till next release
good first issue
hacktoberfest
help wanted
icebox
Invalid
level: 1
level: 2
level: 3
level: 4
needs: exploration
needs: grooming
needs: priority
needs: repro
needs: tech design
on hold
Parked
priority: blocker
priority: high
priority: low
priority: medium
Tom's Wishlist
type: bug
type: discussion
type: improvement
type: new feature
type: refactor
type: task
type: testing
unplanned
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: LBRYCommunity/lighthouse.js#112
Loading…
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?
Examples:
https://lighthouse.lbry.io/search?s=Mar%20GamingYT%20%20%20Robux%20Simulator%20/%20REBIRTH!&size=20&from=0
https://lighthouse.lbry.io/search?s=Mar%20GamingYT%20%20%20Baldi%20Obby!%20/%20I%20got%20lost,%20but%20won!&size=20&from=0
Might be related to some special characters?
Another:
https://lighthouse.lbry.io/search?s=Playing%20Robloxian%20Life%20%20w%20god39lol/%20Mar%20GamingYT&size=20&from=0
The problem is the url is not properly encoded. Forward slash when url encoded should be
%2F
. I think this is the right response when the api receive a malformed url.https://lighthouse.lbry.io/search?s=Mar%20GamingYT%20%20%20RobuxSimulator%20%2F%20REBIRTH
The above is an example where it should work though and does not. The forward slash seems to cause problems somehow.
So I found some more documentation on the escaped characters. Some pages had different characters to escape. I found the official documentation on escaped characters. I noted it in the codebase for reference.
https://www.elastic.co/guide/en/elasticsearch/reference/current/query-dsl-query-string-query.html#_reserved_characters
Solved with https://github.com/lbryio/lighthouse/pull/115