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

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: set correct metric and target fields 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 utils
7 import webapp2 8 import webapp2
8 9
9 from google.appengine.api import users 10 from google.appengine.api import users
10 11
11 12
12 class InternalAlertsHandler(alerts.AlertsHandler): 13 class InternalAlertsHandler(alerts.AlertsHandler):
13 ALERT_TYPE = 'internal-alerts' 14 ALERT_TYPE = 'internal-alerts'
14 15
15 # Has no 'request' member. 16 # Has no 'request' member.
16 # Has no 'response' member. 17 # Has no 'response' member.
17 # Use of super on an old style class. 18 # Use of super on an old style class.
18 # pylint: disable=E1002,E1101 19 # pylint: disable=E1002,E1101
19 def get(self): 20 def get(self):
20 # Require users to be logged to see builder alerts from private/internal 21 # Require users to be logged to see builder alerts from private/internal
21 # trees. 22 # trees.
23 utils.increment_monarch('internal-alerts')
22 user = users.get_current_user() 24 user = users.get_current_user()
23 if not user: 25 if not user:
24 ret = {} 26 ret = {}
25 ret.update({ 27 ret.update({
26 'date': datetime.datetime.utcnow(), 28 'date': datetime.datetime.utcnow(),
27 'redirect-url': users.create_login_url(self.request.uri) 29 'redirect-url': users.create_login_url(self.request.uri)
28 }) 30 })
29 data = self.generate_json_dump(ret) 31 data = self.generate_json_dump(ret)
30 self.send_json_headers() 32 self.send_json_headers()
31 self.response.write(data) 33 self.response.write(data)
32 return 34 return
33 35
34 email = user.email() 36 email = user.email()
35 if not email.endswith('@google.com') and '+' not in email: 37 if not email.endswith('@google.com') and '+' not in email:
36 self.response.set_status(403, 'Permission Denied') 38 self.response.set_status(403, 'Permission Denied')
37 return 39 return
38 40
39 super(InternalAlertsHandler, self).get() 41 super(InternalAlertsHandler, self).get()
40 42
41 43
42 app = webapp2.WSGIApplication([ 44 app = webapp2.WSGIApplication([
43 ('/internal-alerts', InternalAlertsHandler)]) 45 ('/internal-alerts', InternalAlertsHandler)])
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698