search

amilner42 / code-tidbit

Share Programming Knowledge Better
GNU General Public License v3.0
9 stars 1 forks source link

User email contentFilter. #182

Closed amilner42 closed 7 years ago

amilner42 commented 7 years ago

Pure spaces == internal error (cause the userID is in the URL, and express expects something, so /user/bla is fine but /user/ doesn't trigger the same route. Fix by not querying on empty user filters OR not allowing the user to hit space on the first key.

amilner42 commented 7 years ago

Space is not the only issue, I'm realizing it's a poor design to have user input be in the URL like this. For example what if the user input had / or \