openstack/neutron-vpnaas
Code Issues Proposed changes

Fix incorrect exception order in _execute_request

Browse Source 
_execute_request has a list of exception handlers to log various types
of errors with more specific error messages. Unfortunately, it catches
requests.exceptions.ConnectionError before requests.exceptions.SSLError,
but ConnectionError is a superclass of SSLError so the latter is never
invoked.

This change corrects the exception handling order, and enables the
bad-except-order pylint check now that the check passes.

Change-Id: I92bacd6088de5cbc170bc5c081a1db1baeec69e7
Closes-Bug: #1360970
This commit is contained in:
Angus Lees 2014-08-25 12:02:16 +10:00
parent 20d77a868a
commit 43fbb6cf9a
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.pylintrc
View File

@ -19,7 +19,6 @@ disable=
locally-disabled,
# "E" Error for important programming issues (likely bugs)
access-member-before-definition,
bad-except-order,
bad-super-call,
maybe-no-member,
no-member,