| OLD | NEW |
| 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 # Copyright (c) 2012 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 """Defines class Rietveld to easily access a rietveld instance. | 4 """Defines class Rietveld to easily access a rietveld instance. |
| 5 | 5 |
| 6 Security implications: | 6 Security implications: |
| 7 | 7 |
| 8 The following hypothesis are made: | 8 The following hypothesis are made: |
| 9 - Rietveld enforces: | 9 - Rietveld enforces: |
| 10 - Nobody else than issue owner can upload a patch set | 10 - Nobody else than issue owner can upload a patch set |
| 11 - Verifies the issue owner credentials when creating new issues | 11 - Verifies the issue owner credentials when creating new issues |
| 12 - A issue owner can't change once the issue is created | 12 - A issue owner can't change once the issue is created |
| 13 - A patch set cannot be modified | 13 - A patch set cannot be modified |
| 14 """ | 14 """ |
| 15 | 15 |
| 16 import json |
| 16 import logging | 17 import logging |
| 17 import os | |
| 18 import re | 18 import re |
| 19 import sys | |
| 20 import time | 19 import time |
| 21 import urllib2 | 20 import urllib2 |
| 22 | 21 |
| 23 try: | |
| 24 import simplejson as json # pylint: disable=F0401 | |
| 25 except ImportError: | |
| 26 try: | |
| 27 import json # pylint: disable=F0401 | |
| 28 except ImportError: | |
| 29 # Import the one included in depot_tools. | |
| 30 sys.path.append(os.path.join(os.path.dirname(__file__), 'third_party')) | |
| 31 import simplejson as json # pylint: disable=F0401 | |
| 32 | |
| 33 from third_party import upload | 22 from third_party import upload |
| 34 import patch | 23 import patch |
| 35 | 24 |
| 36 # Hack out upload logging.info() | 25 # Hack out upload logging.info() |
| 37 upload.logging = logging.getLogger('upload') | 26 upload.logging = logging.getLogger('upload') |
| 38 # Mac pylint choke on this line. | 27 # Mac pylint choke on this line. |
| 39 upload.logging.setLevel(logging.WARNING) # pylint: disable=E1103 | 28 upload.logging.setLevel(logging.WARNING) # pylint: disable=E1103 |
| 40 | 29 |
| 41 | 30 |
| 42 class Rietveld(object): | 31 class Rietveld(object): |
| (...skipping 300 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 343 if not 'Name or service not known' in e.reason: | 332 if not 'Name or service not known' in e.reason: |
| 344 # Usually internal GAE flakiness. | 333 # Usually internal GAE flakiness. |
| 345 raise | 334 raise |
| 346 # If reaching this line, loop again. Uses a small backoff. | 335 # If reaching this line, loop again. Uses a small backoff. |
| 347 time.sleep(1+maxtries*2) | 336 time.sleep(1+maxtries*2) |
| 348 finally: | 337 finally: |
| 349 upload.ErrorExit = old_error_exit | 338 upload.ErrorExit = old_error_exit |
| 350 | 339 |
| 351 # DEPRECATED. | 340 # DEPRECATED. |
| 352 Send = get | 341 Send = get |
| OLD | NEW |