search

Codrspace / codrspace

The blogging platform for coders.
http://codrspace.com
Other
35 stars 13 forks source link

Don't allow '/' in the title or escape it #34

Closed durden closed 12 years ago

durden commented 12 years ago

I think we could have problems if a user types a '/' character in a title. This will cause 404s I think since the urls.py will match incorrectly.

durden commented 12 years ago

Ok, this seems to work now. I don't see any commits related to it, but I swear I saw a problem with it this morning... Are there any thoughts on this that could cause problems? Maybe some characters we don't escape, etc.?

glenbot commented 12 years ago

I bet so. It sounds reasonable that a specific character would break it. We should sanitize the title and also allow them to modify slug.

glenbot commented 12 years ago

I think this happened because the slug was trimmed. I added slug back to the add/edit pages so this will allow you to adjust your slug if you want to.