listenbrainz-server
listenbrainz-server copied to clipboard
LB-1495: lb-radio endpoint is affected by letter case
Problem
Basically, upper case tag won't do anything. For example: https://api.listenbrainz.org/1/lb-radio/tags?tag=Jazz&begin_percent=0&end_percent=50&count=50 gives 0 results. It's because PG's WHERE clause is case-sensitive.
Solution
Convert all tags into lower case when loading the query params. Also add a integration test for this endpoint.
Hello @ericd23! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Comment last updated at 2024-03-18 03:01:51 UTC
I feel that this isn't a problem that should be fixed in ListenBrainz, its a troi problem. I've fixed this in troi, so closing this PR. Thanks for submitting the PR, much appreciated!