Skip to content

Instantly share code, notes, and snippets.

@tomprince
Created March 5, 2012 15:16
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save tomprince/1978704 to your computer and use it in GitHub Desktop.
Save tomprince/1978704 to your computer and use it in GitHub Desktop.
diff --git a/master/buildbot/status/web/templates/users.html b/master/buildbot/status/web/templates/users.html
index 9c3dafc..f16f119 100644
--- a/master/buildbot/status/web/templates/users.html
+++ b/master/buildbot/status/web/templates/users.html
@@ -5,8 +5,6 @@
<h1>Users</h1>
-{%- if authz.needAuthForm('showUsersPage') -%}
- {{ forms.show_users(table_link, authz) }}
-{% endif %}
+{{ forms.show_users(table_link, authz) }}
{% endblock %}
diff --git a/master/buildbot/status/web/users.py b/master/buildbot/status/web/users.py
index 0a5a574..3284f17 100644
--- a/master/buildbot/status/web/users.py
+++ b/master/buildbot/status/web/users.py
@@ -115,13 +115,12 @@ class UsersResource(HtmlResource):
def content(self, req, ctx):
# check for False or True on showUsersPage, redirect immediately
authz = self.getAuthz(req)
- if not authz.needAuthForm(self.action):
- if authz.advertiseAction(self.action):
- yield redirectTo("users/table", req)
- return
- else:
- yield redirectTo(path_to_authzfail(req), req)
- return
+ if authz.advertiseAction(self.action):
+ yield redirectTo("users/table", req)
+ return
+ else:
+ yield redirectTo(path_to_authzfail(req), req)
+ return
ctx['authz'] = self.getAuthz(req)
ctx['table_link'] = req.childLink("table")
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment