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

Side by Side Diff: appengine/sheriff_o_matic/internal_alerts.py

Issue 1260293009: make version of ts_mon compatible with appengine (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: clean up code Created 5 years, 4 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 unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import datetime 5 import datetime
6 import alerts 6 import alerts
7 import webapp2 7 import webapp2
8 8
9 from google.appengine.api import users 9 from google.appengine.api import users
10 10
11 11
12 class InternalAlertsHandler(alerts.AlertsHandler): 12 class InternalAlertsHandler(alerts.AlertsHandler):
13 ALERT_TYPE = 'internal-alerts' 13 ALERT_TYPE = 'internal-alerts'
14 14
15 # Has no 'request' member. 15 # Has no 'request' member.
16 # Has no 'response' member. 16 # Has no 'response' member.
17 # Use of super on an old style class. 17 # Use of super on an old style class.
18 # pylint: disable=E1002,E1101 18 # pylint: disable=E1002,E1101
19 def get(self): 19 def get(self):
20 # Require users to be logged to see builder alerts from private/internal 20 # Require users to be logged to see builder alerts from private/internal
21 # trees. 21 # trees.
22 alerts.increment_monarch('internal-alerts')
22 user = users.get_current_user() 23 user = users.get_current_user()
23 if not user: 24 if not user:
24 ret = {} 25 ret = {}
25 ret.update({ 26 ret.update({
26 'date': datetime.datetime.utcnow(), 27 'date': datetime.datetime.utcnow(),
27 'redirect-url': users.create_login_url(self.request.uri) 28 'redirect-url': users.create_login_url(self.request.uri)
28 }) 29 })
29 data = self.generate_json_dump(ret) 30 data = self.generate_json_dump(ret)
30 self.send_json_headers() 31 self.send_json_headers()
31 self.response.write(data) 32 self.response.write(data)
32 return 33 return
33 34
34 email = user.email() 35 email = user.email()
35 if not email.endswith('@google.com') and '+' not in email: 36 if not email.endswith('@google.com') and '+' not in email:
36 self.response.set_status(403, 'Permission Denied') 37 self.response.set_status(403, 'Permission Denied')
37 return 38 return
38 39
39 super(InternalAlertsHandler, self).get() 40 super(InternalAlertsHandler, self).get()
40 41
41 42
42 app = webapp2.WSGIApplication([ 43 app = webapp2.WSGIApplication([
43 ('/internal-alerts', InternalAlertsHandler)]) 44 ('/internal-alerts', InternalAlertsHandler)])
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698