Based on the comments and opinions in the original thread, I think a fix for this is justified. I wouldn't mind running this by the TC to double check that nothing has changed from the first time we had to fix this issue though.
On 07/11/2017 06:03 AM, Attila Fazekas wrote: > Hi all, > > Long time ago it was discussed to make the keystone HEAD responses > right [1] as the RFC [2][3] recommends: > > " A response to the HEAD method is identical to what an equivalent > request made with a GET would have been, except it lacks a body. " > > So, the status code needs to be identical as well ! > > Recently turned out, keystone is still not correct in all cases [4]. > > 'Get role inference rule' (GET), 'Confirm role inference rule' (HEAD) > has the same URL pattern, but they differs in the status code (200/204) > which is not allowed! [5] > > This is the only documented case where both the HEAD and GET defined and > the HEAD has a 204 response. > > Are you going to fix this [4] as it was fixed before [6] ? > > Best Regards, > Attila > > PS.: > Here is the tempest change for accepting the right code [7]. > > [1] > http://lists.openstack.org/pipermail/openstack-dev/2014-July/039140.html > [2] https://tools.ietf.org/html/rfc7231#section-4.3.2 > [3] https://tools.ietf.org/html/rfc7234#section-4.3.5 > [4] https://bugs.launchpad.net/keystone/+bug/1701541 > [5] > https://developer.openstack.org/api-ref/identity/v3/?expanded=confirm-role-inference-rule-detail,get-role-inference-rule-detail > [6] https://bugs.launchpad.net/keystone/+bug/1334368 > [7] https://review.openstack.org/#/c/479286/ > > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: [email protected]?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
signature.asc
Description: OpenPGP digital signature
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
