Closed
Bug 1089726
Opened 10 years ago
Closed 10 years ago
Remove any control characters in vidly status cache key
Categories
(Webtools Graveyard :: Air Mozilla, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: peterbe, Unassigned)
References
()
Details
(Whiteboard: [good first bug])
When we make the cache key [0] we blindly use the tag value of the event. If this contains anything other that [a-z][0-9] it can cause trouble (memcache trouble). We should do something like: tag = environment['tag'] cache_key = 'vidly-query-%s' % hashlib.md5(tag).hexdigest() [0] https://github.com/mozilla/airmozilla/blob/810ead7498fc54895d36c5917d7dec771c835a93/airmozilla/manage/views.py#L2350
|
Reporter | |
Updated•10 years ago
|
Whiteboard: [good first bug]
|
|
Reporter | |
Updated•10 years ago
|
|
||
Comment 1•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/airmozilla https://github.com/mozilla/airmozilla/commit/d277a791de583775f77fa80cff491f057bd2deb0 Fix bug 1089726 by md5-ing cache key. https://github.com/mozilla/airmozilla/commit/06a2960bfb92ec5b534c6fb167d7177ec43d3d86 Merge pull request #120 from koddsson/md5-memcached-key Fix bug 1089726 by md5-ing cache key.
|
|
||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
|
||
Updated•3 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•