opa icon indicating copy to clipboard operation
opa copied to clipboard

Forward incoming HTTP Request ID to remote services when using http.send

Open kotyara85 opened this issue 1 year ago • 6 comments

It looks like OPA doesn't honor x-request-id header at this moment It would be great if it did, because that would give an ability to run E2E traces in case if http.send is involved Thanks

kotyara85 avatar Aug 12 '22 18:08 kotyara85

Can you please elaborate on your expectations? Should the request ID header become part of the response, or part of the open telemetry spans, or be used in decision logs...? What is you're checking in your E2E traces?

srenatus avatar Aug 12 '22 18:08 srenatus

Yeah, for example: serviceA -> OPA -> serviceB

serviceA adds x-request-id header this header would be exposed as reqId in opa info log and passed down to serviceB as x-request-id header

This way you can see one call across all the systems

Thanks

kotyara85 avatar Aug 12 '22 18:08 kotyara85

Ah. So concretely, you're expecting the header to be sent along with http.send calls to remote services.

srenatus avatar Aug 12 '22 18:08 srenatus

Ah. So concretely, you're expecting the header to be sent along with http.send calls to remote services.

Yeah

kotyara85 avatar Aug 12 '22 18:08 kotyara85

@srenatus it would be also beneficial to add reqId to logs

kotyara85 avatar Aug 18 '22 02:08 kotyara85

@kotyara85 did you check out the open telemetry integration?

anderseknert avatar Aug 18 '22 11:08 anderseknert

This issue has been automatically marked as inactive because it has not had any activity in the last 30 days.

stale[bot] avatar Oct 02 '22 00:10 stale[bot]