Merge lp:~ben-hutchings/ensoft-sextant/audit-error-fix into lp:ensoft-sextant

Proposed by Ben Hutchings
Status: Merged
Approved by: Robert
Approved revision: 25
Merged at revision: 25
Proposed branch: lp:~ben-hutchings/ensoft-sextant/audit-error-fix
Merge into: lp:ensoft-sextant
Diff against target: 13 lines (+2/-1)
1 file modified
src/sextant/__main__.py (+2/-1)
To merge this branch: bzr merge lp:~ben-hutchings/ensoft-sextant/audit-error-fix
Reviewer Review Type Date Requested Status
Phil Connell Approve
Robert Approve
Review via email: mp+236101@code.launchpad.net

Commit message

Fix the error message shown when sextant fails to connect to the server (was previously throwing an exception due to a misplaced bracket).

Description of the change

Fixed the error message shown when sextant fails to connect to the server (was previously throwing an exception due to a misplaced bracket).

To post a comment you must log in.
Revision history for this message
Robert (rjwills) :
review: Approve
Revision history for this message
Phil Connell (pconnell) wrote :

Adding 'approve' review so this actually gets merged by tarmac :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/sextant/__main__.py'
--- src/sextant/__main__.py 2014-09-02 11:03:34 +0000
+++ src/sextant/__main__.py 2014-09-26 11:38:25 +0000
@@ -74,7 +74,8 @@
74 audited = query.audit(args.remote_neo4j)74 audited = query.audit(args.remote_neo4j)
75 except requests.exceptions.ConnectionError as e:75 except requests.exceptions.ConnectionError as e:
76 msg = 'Connection error to server {url}: {exception}'76 msg = 'Connection error to server {url}: {exception}'
77 logging.error(msg.format(url=_displayable_url(args)), exception=e)77 logging.error(msg.format(url=_displayable_url(args), exception=e))
78 return
7879
79 if not audited:80 if not audited:
80 location = _displayable_url(args)81 location = _displayable_url(args)

Subscribers

People subscribed via source and target branches