HEAD request over a container is not returning the ACL headers

Bug #879371 reported by Juan J. Martínez
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Invalid
Undecided
Juan J. Martínez

Bug Description

I've noticed this problem with 1.4.3, although I think it was introduced by the container sync code.

Steps to reproduce:

 1. Set read and write ACL in a container
 2. Get the container information

There should be two headers X-Container-Read and X-Container-Write (as in 1.4.0), but they're not present.

Please see attached test script.

Revision history for this message
Juan J. Martínez (jjmartinez) wrote :
Revision history for this message
Juan J. Martínez (jjmartinez) wrote :

The code introducing the bug seems to be in swift/server/proxy.py:1288, that removes the x-container-read and x-container-write headers, thus breaking the HEAD request (get container info).

These headers should persist. What do you think?

I'm attaching a patch that solves the problem, but I don't know id there's some kind of side effect.

Revision history for this message
Juan J. Martínez (jjmartinez) wrote :

I can reproduce the problem in current code, a HEAD request over a container doesn't include X-Container-Read and X-Container-Write in the response.

This change of behaviour appeared in 1.4.3. I can't find information reasoning this change.

Is this a bug or an intended change of behaviour?

Changed in swift:
status: New → Confirmed
assignee: nobody → Juan J. Martínez (jjmartinez)
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/3656

Changed in swift:
status: Confirmed → In Progress
Changed in swift:
status: In Progress → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

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