Opened 17 years ago
Closed 17 years ago
#5557 closed (fixed)
The cache_page decorator fails on urls containing spaces
Reported by: | Owned by: | Jacob | |
---|---|---|---|
Component: | Core (Cache system) | Version: | dev |
Severity: | Keywords: | cache_page | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Memcache raises an exception when a key contains a space. The cache_page decorator does not escape the url of the page causing a server error.
The included patch wraps the use of request.path with a call to uri_to_iri()
Attachments (1)
Change History (5)
by , 17 years ago
Attachment: | django-utils-cache-py.patch added |
---|
comment:1 by , 17 years ago
This error also occurs when memcached tries to caché an url item that gives 404 error. Django tries to generate a key with the prefix, the url and the error message.
views.decoratore.cache_header.tc./trobacasa/promocion/1/Error obrint la imatge /home/aaloy/wor/apsl/media/promociones/2006/12/04/2-foto.jpg
comment:2 by , 17 years ago
Triage Stage: | Unreviewed → Accepted |
---|
We've run into this a couple times at World Online, and used various workarounds. It'd be nice to solve it once and for all in Django's caching system.
comment:3 by , 17 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:4 by , 17 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Proposed patch