-
Notifications
You must be signed in to change notification settings - Fork 62
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
[API-39196] Add transaction_id
to 526 synchronous logs
#18210
Conversation
modules/claims_api/app/controllers/claims_api/v2/veterans/disability_compensation_controller.rb
Outdated
Show resolved
Hide resolved
modules/claims_api/app/services/claims_api/disability_compensation/docker_container_service.rb
Show resolved
Hide resolved
modules/claims_api/app/services/claims_api/disability_compensation/pdf_generation_service.rb
Show resolved
Hide resolved
modules/claims_api/app/services/claims_api/disability_compensation/pdf_generation_service.rb
Show resolved
Hide resolved
modules/claims_api/app/services/claims_api/disability_compensation/pdf_generation_service.rb
Show resolved
Hide resolved
23e7b21
to
8fd12e6
Compare
8fd12e6
to
3c80819
Compare
3c80819
to
790e20b
Compare
790e20b
to
98d4f30
Compare
Add `transaction_id` to: * Docker container service logs * PDF generation service logs * EVSS service logs
98d4f30
to
fa78d17
Compare
modules/claims_api/app/services/claims_api/disability_compensation/docker_container_service.rb
Show resolved
Hide resolved
modules/claims_api/app/services/claims_api/disability_compensation/docker_container_service.rb
Show resolved
Hide resolved
modules/claims_api/app/services/claims_api/disability_compensation/pdf_generation_service.rb
Show resolved
Hide resolved
modules/claims_api/app/services/claims_api/disability_compensation/pdf_generation_service.rb
Show resolved
Hide resolved
Hey @tycol7, I am not seeing the transaction_id come through in the logs. |
Did your |
Got it, my apologies. |
Backend-review-group approval confirmed. |
Summary
Add
transaction_id
to:Related issue(s)
API-39196
Testing done
Screenshots
N/A
What areas of the site does it impact?
526 synchronous endpoint, including:
Acceptance criteria
Requested Feedback
N/A