darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-03 07:31:28 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2231 from alphagov/zipkin-headers

Browse Source 
Both trace_id and span_id headers must be present
This commit is contained in:
Sakis
2018-08-15 16:10:09 +01:00
committed by GitHub
parent 99e2f481c8 984e09cc6a
commit 85ec7a28cb
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
tests/app/notify_client/test_notify_admin_api_client.py
View File

@@ -99,6 +99,7 @@ def test_generate_headers_sets_request_id_if_in_request_context(app_):
headers = api_client.generate_headers('api_token')
assert set(headers.keys()) == {
'Authorization', 'Content-type', 'User-agent', 'X-Custom-Forwarder', 'X-B3-TraceId'
'Authorization', 'Content-type', 'User-agent', 'X-Custom-Forwarder', 'X-B3-TraceId', 'X-B3-SpanId',
}
assert headers['X-B3-TraceId'] == request_context.request.request_id
assert headers['X-B3-SpanId'] == request_context.request.span_id