bad accept header causes 503

Bug #891247 reported by Scott Simpson on 2011-11-16
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Undecided
Scott Simpson
Essex
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

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) on 2012-01-05
Changed in swift:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers