andreisavu / django-jack

Jack and the Beanstalkd. Webapp for basic work queue administration.
http://kr.github.com/beanstalkd/
Apache License 2.0
60 stars 12 forks source link

slashes (/) in tube names #4

Open drasch opened 13 years ago

drasch commented 13 years ago

We use slashes in our tube names and they look like paths. This causes issues with the urls regex for the tube_stats view. I fixed this by adding \/ to the regex before the -

diff --git a/jack/beanstalk/urls.py b/jack/beanstalk/urls.py
index 51cbe4c..4e5c9a9 100644
--- a/jack/beanstalk/urls.py
+++ b/jack/beanstalk/urls.py
@@ -6,11 +6,11 @@ urlpatterns = patterns('',
     (r'^$', views.index),
     (r'^stats/$', views.stats),
     (r'^put/$', views.put),
-    (r'^ready/(?P<tube>[\w-]*)$', views.ready),
-    (r'^delayed/(?P<tube>[\w-]*)$', views.delayed),
-    (r'^buried/(?P<tube>[\w-]*)$', views.buried),
+    (r'^ready/(?P<tube>[\w\/-]*)$', views.ready),
+    (r'^delayed/(?P<tube>[\w\/-]*)$', views.delayed),
+    (r'^buried/(?P<tube>[\w\/-]*)$', views.buried),
     (r'^inspect/(?P<id>\d*)$', views.inspect),
-    (r'^tube/(?P<tube>[\w-]+)/stats/$', views.tube_stats),
+    (r'^tube/(?P<tube>[\w\/-]+)/stats/$', views.tube_stats),
     (r'^job/(?P<id>\d+)/delete/$', views.job_delete),
     (r'^job/(?P<id>\d+)/kick/$', views.job_kick),
 )

Thanks, David