Re: Issue 2538 in reviewboard: Djblets: memcached leakage across instances - when instances run same server domain but different site root

2012-09-28 Thread reviewboard

Updates:
Status: Fixed

Comment #2 on issue 2538 by trowb...@gmail.com: Djblets: memcached leakage  
across instances - when instances run same server domain but different site  
root

http://code.google.com/p/reviewboard/issues/detail?id=2538

Thanks! Pushed to master (0fdc48b)

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Re: Issue 2538 in reviewboard: Djblets: memcached leakage across instances - when instances run same server domain but different site root

2012-03-20 Thread reviewboard


Comment #1 on issue 2538 by e...@tibco.com: Djblets: memcached leakage  
across instances - when instances run same server domain but different site  
root

http://code.google.com/p/reviewboard/issues/detail?id=2538

I believe the following patch does the job of fixing this issue against the  
Djblets project:



diff --git a/djblets/util/misc.py b/djblets/util/misc.py
index 5fb8da8..521bfab 100644
--- a/djblets/util/misc.py
+++ b/djblets/util/misc.py
@@ -193,7 +193,12 @@ def make_cache_key(key):
 site = Site.objects.get_current()

 # The install has a Site app, so prefix the domain to the key.
-key = "%s:%s" % (site.domain, key)
+# Also prefix the site root, if there is one, to allow for
+# multiple instances on the same host.
+if settings.SITE_ROOT:
+key = "%s:%s:%s" % (site.domain, settings.SITE_ROOT, key)
+else:
+key = "%s:%s" % (site.domain, key)
 except:
 # The install doesn't have a Site app, so use the key as-is.
 pass


--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.



Issue 2538 in reviewboard: Djblets: memcached leakage across instances - when instances run same server domain but different site root

2012-03-20 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 2538 by e...@tibco.com: Djblets: memcached leakage across  
instances - when instances run same server domain but different site root

http://code.google.com/p/reviewboard/issues/detail?id=2538

As per this email thread:

http://groups.google.com/group/reviewboard/browse_thread/thread/2f6b51a052271624#

I've noticed some "cache leakage" from one reviewboard instance. I host  
multiple reviewboard instances on one server, and they all share the same  
domain name for the host, but different "SITE_ROOT"s.



--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.