Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2137)

Unified Diff: appengine/chromium_cq_status/handlers/stats_viewer.py

Issue 2111713004: Teach CQ status app to check login status of users. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: cleanup Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: appengine/chromium_cq_status/handlers/stats_viewer.py
diff --git a/appengine/chromium_cq_status/handlers/stats_viewer.py b/appengine/chromium_cq_status/handlers/stats_viewer.py
index df21b2b3fc497d6f72473bbf73f882fdb996a708..284d7c19e7c36bb1ce30fb06d6b91dec518967ce 100644
--- a/appengine/chromium_cq_status/handlers/stats_viewer.py
+++ b/appengine/chromium_cq_status/handlers/stats_viewer.py
@@ -4,7 +4,7 @@
import webapp2
-from shared.utils import minutes_per_day
+from shared.utils import minutes_per_day, read_access
Sergiy Byelozyorov 2016/07/01 17:13:06 for consistency with other files, please rewrite a
tandrii(chromium) 2016/07/01 18:56:55 Done.
period_config = {
'15-minutely': {
@@ -30,6 +30,7 @@ period_config = {
}
class StatsViewer(webapp2.RequestHandler): # pragma: no cover
+ @read_access
def get(self, project, period):
assert period in period_config
config = period_config[period]

Powered by Google App Engine
This is Rietveld 408576698