Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use correlation id from butler-audit when logging to Graylog. #49

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

emil-nasso
Copy link
Contributor

Use the correlation-id published by butler-audit when logging
to graylog and rename the extra field to correlation_id to
follow the same naming convention.

This change is made to be able to trace requests between systems.

@emil-nasso emil-nasso requested review from wecc and ttrig January 14, 2021 08:30
@emil-nasso emil-nasso force-pushed the cczb4a-graylog-correlation-id branch from 7c91bed to c4d7406 Compare January 14, 2021 08:38
@ttrig
Copy link
Contributor

ttrig commented Jan 18, 2021

@emil-nasso emil-nasso force-pushed the cczb4a-graylog-correlation-id branch from c4d7406 to ca005f7 Compare January 21, 2021 08:51
@emil-nasso emil-nasso changed the title Use correlation id from butler-auth when logging to Graylog. draft: Use correlation id from butler-auth when logging to Graylog. Feb 2, 2021
@emil-nasso emil-nasso changed the title draft: Use correlation id from butler-auth when logging to Graylog. Use correlation id from butler-auth when logging to Graylog. Feb 2, 2021
@emil-nasso emil-nasso marked this pull request as draft February 2, 2021 09:31
@emil-nasso emil-nasso force-pushed the cczb4a-graylog-correlation-id branch from ca005f7 to f708f00 Compare February 24, 2021 07:57
@wecc wecc changed the title Use correlation id from butler-auth when logging to Graylog. Use correlation id from butler-audit when logging to Graylog. Feb 24, 2021
@emil-nasso emil-nasso force-pushed the cczb4a-graylog-correlation-id branch from f708f00 to c4c2c66 Compare February 24, 2021 08:41
@emil-nasso emil-nasso marked this pull request as ready for review February 24, 2021 08:45
@ttrig
Copy link
Contributor

ttrig commented Feb 25, 2021

  • changelog

@emil-nasso emil-nasso force-pushed the cczb4a-graylog-correlation-id branch 2 times, most recently from 3e4ca9c to ad58913 Compare March 1, 2021 15:01
Use the correlation-id published by butler-audit when logging
to graylog to allow for grouping all graylog entries originating
from the same request.

This change is made to be able to trace requests between systems.
@emil-nasso emil-nasso force-pushed the cczb4a-graylog-correlation-id branch from ad58913 to 00bc7a5 Compare March 1, 2021 15:03
@emil-nasso emil-nasso merged commit 96897ac into master Mar 11, 2021
@ttrig ttrig deleted the cczb4a-graylog-correlation-id branch March 18, 2021 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants