⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@iamajais
Copy link
Contributor

No description provided.

@iamajais iamajais requested a review from a team as a code owner January 12, 2026 11:31
@github-actions
Copy link

Test Results

90 tests  ±0   90 ✅ ±0   25s ⏱️ -1s
16 suites ±0    0 💤 ±0 
16 files   ±0    0 ❌ ±0 

Results for commit 25653af. ± Comparison against base commit dc4df4c.

@iamajais iamajais merged commit 3e82918 into main Jan 12, 2026
5 checks passed
@iamajais iamajais deleted the feature/distributed-logging branch January 12, 2026 11:37
public static final Set<String> HEADER_PREFIXES_TO_BE_PROPAGATED =
Set.of(
TENANT_ID_HEADER_KEY,
CONTEXT_ID_HEADER_KEY,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's context ID and how does it differ from request ID?

Copy link
Contributor Author

@iamajais iamajais Jan 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

context id will be kept same for all requests for a particular context i.e scan context. Depends on the individual client what they want to use as context-id. For our use case we will use scanId as context-id so that we can tail all the logs for a particular scan. We kept it different from request-id so that we can differentiate between multiple requests.

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.

5 participants