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

Specify use of trace context #150

Merged
merged 2 commits into from
Mar 5, 2024
Merged

Specify use of trace context #150

merged 2 commits into from
Mar 5, 2024

Conversation

qligier
Copy link
Collaborator

@qligier qligier commented Feb 5, 2024

@qligier qligier marked this pull request as draft February 5, 2024 18:56
@qligier
Copy link
Collaborator Author

qligier commented Feb 20, 2024

Consecutive HTTP requests SHALL NOT have the same trace-id -> Clients SHALL use a different trace-id for each new request

Volume 1/Swiss requirements -> Appendix/Trace Context

Remove section "Authorization header"

Also include ITI-20 in scope.

Replace "EPR platform requirements" with:
Each actor shall support the traceparent header. Grouped actors shall use the same trace-id value to correlate IHE transactions.

EPR platform -> responder

@qligier qligier force-pushed the ql_trace_context branch 5 times, most recently from b930981 to 15e3ef2 Compare February 23, 2024 09:48
@qligier qligier marked this pull request as ready for review February 23, 2024 12:56
@qligier
Copy link
Collaborator Author

qligier commented Feb 23, 2024

Updated as discussed.

BALP PR: IHE/ITI.BasicAudit#100

@oliveregger
Copy link
Collaborator

@qligier
Copy link
Collaborator Author

qligier commented Feb 29, 2024

IHE/ITI.MHD#220

@qligier
Copy link
Collaborator Author

qligier commented Feb 29, 2024

@oliveregger oliveregger merged commit c4c0157 into master Mar 5, 2024
@qligier qligier deleted the ql_trace_context branch March 12, 2024 13:59
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.

enforce trace context in http header
2 participants