OpenStack Object Storage (swift)

Merge lp://staging/~david-goetz/swift/invalid_url_bug into lp://staging/~hudson-openstack/swift/trunk

  1. invalid_url_bug
  2. Merge into trunk
Proposed by David Goetz
Status: Merged
Approved by: Chuck Thier
Approved revision: 115
Merged at revision: 114
Proposed branch: lp://staging/~david-goetz/swift/invalid_url_bug
Merge into: lp://staging/~hudson-openstack/swift/trunk
Diff against target: 251 lines (+76/-17)
11 files modified
swift/account/server.py (+6/-1)
swift/auth/server.py (+5/-2)
swift/common/middleware/auth.py (+12/-6)
swift/common/middleware/ratelimit.py (+5/-1)
swift/common/utils.py (+2/-0)
swift/obj/server.py (+6/-2)
swift/proxy/server.py (+2/-0)
swift/stats/access_processor.py (+7/-4)
test/functional/tests.py (+9/-0)
test/unit/auth/test_server.py (+2/-1)
test/unit/common/middleware/test_ratelimit.py (+20/-0)
To merge this branch: bzr merge lp://staging/~david-goetz/swift/invalid_url_bug
Reviewer Review Type Date Requested Status
Chuck Thier (community) Approve
Review via email: mp+39660@code.staging.launchpad.net

Description of the change

catching invalid urls and adding tests

To post a comment you must log in.
Revision history for this message
Chuck Thier (cthier) wrote :

Looks good to me

review: Approve
Revision history for this message
Chuck Thier (cthier) wrote :

btw, I added another note to the ticket for his issue so that we can track back porting it to the 1.1 series.

Revision history for this message
Chuck Thier (cthier) wrote :

Ok looks good now as well, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.