bad accept header causes 503

Bug #891247 reported by Scott Simpson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Fix Released
Undecided
Scott Simpson
Essex
Fix Released
Undecided
Scott Simpson

Bug Description

bad accept headers can cause 503 errors instead of a 400 Bad Request. headers like accept: foo/text* instead of foo/* will cause webob to throw an AssertionError.

Changed in swift:
assignee: nobody → Scott Simpson (sasimpson)
Changed in swift:
status: New → Fix Committed
Changed in swift:
status: Fix Committed → In Progress
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to swift (master)

Reviewed: https://review.openstack.org/1701
Committed: http://github.com/openstack/swift/commit/dfb9a9f0a340d3a1c4881f7a61d44bada1002979
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit dfb9a9f0a340d3a1c4881f7a61d44bada1002979
Author: Scott Simpson <email address hidden>
Date: Wed Nov 16 13:48:23 2011 -0600

    fixes bug lp 891247

    added try/except around the accept header matcher
    to catch the assertionerror that was being thrown
    and return a more meaningful message.

    Change-Id: I64184be0a40f8696f8e7e3801763d555ec2526dd

Changed in swift:
status: In Progress → Fix Committed
Changed in swift:
milestone: none → 1.4.5
Thierry Carrez (ttx)
Changed in swift:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.