18b8033b6b
This patch fixes multiple issues related to request ids: - Duplicated request ID in logs - Missing request id header in response - Wrong request ID in response header when using noauth The source of these issues is the fact that the request ID and the request context are set too late, or not at all, in the paste filter chain. And the last bug is fixed as a freebie when the code in auth.py is refactored to reduce code duplication on setting the cinder.context key in the request's environment. The patch fixes these issue by changing the filter chains to have the request_id filter as the first filter, and using our own filter for it, to ensure that the context is correctly set. Not sure if values now defined in ENV_OVERWRITES are still necessary, so the patch preserves current behavior. Closes-Bug: #1960019 Closes-Bug: #1960020 Closes-Bug: #1960021 Closes-Bug: #1960329 Change-Id: Ide831a221a7561bbb7da193f0edb011f1989420e
16 lines
667 B
YAML
16 lines
667 B
YAML
---
|
|
fixes:
|
|
- |
|
|
`Bug #1960019 <https://bugs.launchpad.net/cinder/+bug/1960019>`_: Fixed
|
|
value of the x-openstack-request-id header when Cinder is using noauth.
|
|
- |
|
|
`Bug #1960020 <https://bugs.launchpad.net/cinder/+bug/1960020>`_: Fixed
|
|
duplicated request-id values in logs for different requests, happens only
|
|
on request to / to get available api versions.
|
|
- |
|
|
`Bug #1960021 <https://bugs.launchpad.net/cinder/+bug/1960021>`_: Fixed
|
|
missing request id headers in requests to / to get available api versions.
|
|
- |
|
|
`Bug #1960329 <https://bugs.launchpad.net/cinder/+bug/1960329>`_: Fixed
|
|
wrong request ID on middleware filters.
|