Merge "Added request_id and global_request_id to CADF notifications"

This commit is contained in:
Zuul 2018-12-12 21:58:49 +00:00 committed by Gerrit Code Review
commit ac7c7fba0a
3 changed files with 37 additions and 2 deletions

View File

@ -57,10 +57,15 @@ class PycadfAuditApiConfigError(Exception):
class ClientResource(resource.Resource):
def __init__(self, project_id=None, **kwargs):
def __init__(self, project_id=None, request_id=None,
global_request_id=None, **kwargs):
super(ClientResource, self).__init__(**kwargs)
if project_id is not None:
self.project_id = project_id
if request_id is not None:
self.request_id = request_id
if global_request_id is not None:
self.global_request_id = global_request_id
class KeystoneCredential(credential.Credential):
@ -297,7 +302,10 @@ class OpenStackAuditApi(object):
token=req.environ.get('HTTP_X_AUTH_TOKEN', ''),
identity_status=req.environ.get('HTTP_X_IDENTITY_STATUS',
taxonomy.UNKNOWN)),
project_id=req.environ.get('HTTP_X_PROJECT_ID', taxonomy.UNKNOWN))
project_id=req.environ.get('HTTP_X_PROJECT_ID', taxonomy.UNKNOWN),
request_id=req.environ.get('openstack.request_id'),
global_request_id=req.environ.get('openstack.global_request_id'))
target = self.get_target_resource(req)
event = factory.EventFactory().new_event(

View File

@ -395,3 +395,25 @@ class AuditApiLogicTest(base.BaseAuditMiddlewareTest):
self.assertNotIn('reporterchain', payload)
self.assertEqual(payload['observer']['id'], 'target')
self.assertEqual(path, payload['requestPath'])
def test_request_and_global_request_id(self):
path = '/v1/' + str(uuid.uuid4())
url = 'https://23.253.72.207' + path
request_id = 'req-%s' % uuid.uuid4()
global_request_id = 'req-%s' % uuid.uuid4()
env_headers = self.get_environ_header('GET')
env_headers['openstack.request_id'] = request_id
env_headers['openstack.global_request_id'] = global_request_id
payload = self.get_payload('GET', url, environ=env_headers)
self.assertEqual(payload['initiator']['request_id'], request_id)
self.assertEqual(payload['initiator']['global_request_id'],
global_request_id)
payload = self.get_payload('GET', url)
self.assertNotIn('request_id', payload['initiator'])
self.assertNotIn('global_request_id', payload['initiator'])

View File

@ -0,0 +1,5 @@
---
features:
- >
[`bug 1803940 <https://bugs.launchpad.net/keystonemiddleware/+bug/1803940>`_]
Request ID and global request ID have been added to CADF notifications.