segler-alex / radiobrowser-api-rust

radio-browser API implementation in rust
GNU Affero General Public License v3.0
234 stars 104 forks source link

Tens of thousands of unnecessary characters in some streaming links #154

Closed GHbasicuser closed 1 year ago

GHbasicuser commented 2 years ago

Hi,

I noticed that there were many radio station links in the database that contained more than 14,000 totally useless (extra) characters. (like this one "Tick Tock Radio - 1950" - https://www.radio-browser.info/history/cde988ac-9376-41c7-a29c-ae779cbb5830 - 14.000 "O" and a F* y.. at the end)

Most seem to be "Tick Tock" (IceCast Audio) radio stations. Wouldn't there be a way to prevent this and clean up the base?

Thank you, Good continuation !

GHbasicuser commented 2 years ago

This link isn't broken, it contains thousands of extra characters (totally useless) but it's functional. 😐

Explosion-Scratch commented 2 years ago

What about using a URL shortener for url_resolved if it is over a certain length? I know that is.gd has a public no limits no auth api for this

JazzfanRS commented 2 years ago

I have simply decided to strip all URL parameters (anything after the question mark) as they aren't needed to listen to stream.