Site Names in URL - Githubissues

agrc / land-mobile-radio

View application for looking at combined service areas of 800MHz Land Mobile Radio Towers

MIT License

0 stars 0 forks source link

Site Names in URL #17

Open brigsz opened 9 years ago

brigsz commented 9 years ago

Site names instead of IDs in the URL parameter

stdavis commented 9 years ago

Site names worried me for two reasons:

They might contain funny characters that would not be handled correctly in the URL.
They are much longer than the ID's which could cause the URL to be longer than the recommended 2000 character limit.

These are just concerns and if it's really important to have the site names and if they think they won't hit the 2000 limit then I'm fine with changing it.

stdavis commented 9 years ago

There's also some optimization that I could to do eliminate the query parameter name repeating for each value that would help with the URL length.

2 hours