Swift tests for unicode headers

Bug #1189123 reported by David Hadas
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Fix Released
Undecided
David Hadas

Bug Description

Swift tests added in April 24, includes a test that sends unicode headers to Swift.

The tests expects the header to be accepted.
Unicode headers are not according to the HTTP standard.
Apache2 seem to remove such headers which is a sound behavior for illegal headers.

The tests should be changed to either not send unicode headers or at least restrict such tests to swift integral web front end only.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to swift (master)

Fix proposed to branch: master
Review: https://review.openstack.org/32315

Changed in swift:
assignee: nobody → David Hadas (david-hadas)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to swift (master)

Reviewed: https://review.openstack.org/32315
Committed: http://github.com/openstack/swift/commit/6d59cbf596589b922e33969580321198f2482700
Submitter: Jenkins
Branch: master

commit 6d59cbf596589b922e33969580321198f2482700
Author: David Hadas <email address hidden>
Date: Sun Jun 9 12:54:32 2013 +0300

    Swift tests for unicode headers

    Swift functional tests sends ilegal headers (unicode) and expects
    these headers to be processed by Swift.

    Apache2 seem to remove such headers which is a sound behavior for
    illegal headers.

    This change restrict the test tpo exist only for swift integral
    web front end. For future it is advised to disallow such behaviour.
    For example it is suggested to make the decission that in future APIs
    illegal headers are removed rather than processed.

    Change-Id: Ibf548f474de3f06a2687f8f4812beaeaf81ab216
    Fixes: Bug #1189123

Changed in swift:
status: In Progress → Fix Committed
Changed in swift:
milestone: none → 1.9.0
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.