adt-cloud-service

Merge lp:~joetalbott/adt-cloud-service/fix_reverse_logic_for_trailing_slash into lp:adt-cloud-service

  1. fix_reverse_logic_for_trailing_slash
  2. Merge into trunk
Proposed by Joe Talbott
Status: Merged
Approved by: Celso Providelo
Approved revision: 28
Merged at revision: 25
Proposed branch: lp:~joetalbott/adt-cloud-service/fix_reverse_logic_for_trailing_slash
Merge into: lp:adt-cloud-service
Diff against target: 12 lines (+1/-1)
1 file modified
adt_cloud_service/v1.py (+1/-1)
To merge this branch: bzr merge lp:~joetalbott/adt-cloud-service/fix_reverse_logic_for_trailing_slash
Reviewer Review Type Date Requested Status
Celso Providelo (community) Approve
Review via email: mp+256571@code.launchpad.net

Commit message

Fix reversed logic for trailing slash check.

Description of the change

Fix reversed logic for trailing slash check.

To post a comment you must log in.
Revision history for this message
Ubuntu CI Bot (uci-bot) wrote :

Voting does not meet specified criteria. Required: Approve >= 1. Got: 1 Pending.

Revision history for this message
Celso Providelo (cprov) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'adt_cloud_service/v1.py'
2--- adt_cloud_service/v1.py 2015-04-16 18:35:43 +0000
3+++ adt_cloud_service/v1.py 2015-04-16 21:21:22 +0000
4@@ -61,7 +61,7 @@
5 req_id = current_app.extensions['rabbit'].queue_test(rabbit_request)
6 results_path = 'adt-{}/results.tgz'.format(req_id)
7 swift_url = current_app.config['SWIFT_URL']
8- if swift_url.endswith('/'):
9+ if not swift_url.endswith('/'):
10 swift_url += "/"
11 return jsonify(
12 {

Subscribers

People subscribed via source and target branches