Skip to content
This repository has been archived by the owner on Oct 8, 2023. It is now read-only.

UUID format uppercase/lowercase #147

Closed
NerfBro opened this issue Apr 15, 2022 · 1 comment
Closed

UUID format uppercase/lowercase #147

NerfBro opened this issue Apr 15, 2022 · 1 comment
Assignees
@segler-alex
Labels
bug Something isn't working
Milestone
0.7.23

Comments

@NerfBro
Copy link

NerfBro commented Apr 15, 2022

I've noticed that the following call will fail if the UUIDs are in capitals. It took me many hours to figure this out...
https://all.api.radio-browser.info/json/stations/byuuid?uuids=d873afb3-277f-4d42-9b36-ced320409734,9615d27a-0601-11e8-ae97-52543be04c81

I know the standard is that they be lowercase, but to make the API more compatible with the most frameworks could this call be made non case sensitive?
@segler-alex segler-alex self-assigned this Apr 17, 2022
@segler-alex segler-alex added this to the 0.7.23 milestone Apr 17, 2022
@segler-alex
Copy link
Owner

i will do a conversion to a default lower case on this input in the next version

@segler-alex segler-alex added the bug Something isn't working label Apr 17, 2022
segler-alex added a commit that referenced this issue Apr 17, 2022
@segler-alex
lowercase uuid, Closes #147
a0737f0
segler-alex added a commit that referenced this issue Apr 17, 2022
@segler-alex
convert all uuids to lowercase in params, #147
c551ccf
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@segler-alex segler-alex

Labels
bug Something isn't working
Projects
None yet
Milestone
0.7.23
Development

No branches or pull requests
2 participants
@segler-alex @NerfBro