segler-alex / radiobrowser-api-rust

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

Station naming #153

Closed NerfBro closed 1 year ago

NerfBro commented 2 years ago

Hi,

I've spent a fair bit of time going through all the stations and have found many that have very spurrious names. Some start with whitepace chaacters such as newline or tabs. Other have clever names starting with *** or __ or --- etc. I've also found many that are over 40 characters long, even a dozen or so starting with 'http'. In short, it's somewhat inconsistent.

I propose adding some restrictions on how these names are entered in to the DB and sanitise them at that point. Names should never contain tabs or newline chars and they should all be less than 40 chars in length IMHO (if people want to describe the station we really need a separate field for that information).

Thanks.