Skip to content

feat: added support for w3c sampled flag#2512

Open
kirrg001 wants to merge 5 commits intomainfrom
feat-w3c-flag
Open

feat: added support for w3c sampled flag#2512
kirrg001 wants to merge 5 commits intomainfrom
feat-w3c-flag

Conversation

@kirrg001
Copy link
Copy Markdown
Contributor

@kirrg001 kirrg001 commented Apr 22, 2026

verifyHttpHeadersOnDownstreamRequest(testDefinition, valuesForPlaceholders, responseBody);

const suppressed = testDefinition['X-INSTANA-L in'] === '0';
const suppressed = testDefinition.suppressed === true;
Copy link
Copy Markdown
Contributor Author

@kirrg001 kirrg001 Apr 23, 2026

Choose a reason for hiding this comment

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

The expected output is now defined in the JSON files.
Checking X-INSTANA-L is no longer enough, we need to also test if no X-INSTANA-L value is received.

@kirrg001 kirrg001 marked this pull request as ready for review April 23, 2026 09:43
@kirrg001 kirrg001 requested a review from a team as a code owner April 23, 2026 09:43
// upate it. In case we received X-INSTANA-L: 0 we will not start a span, but we will make sure to toggle the
// sampled flag in traceparent off.

// CASE: Respect W3C sampled flag (only if X-INSTANA-L header is not present, as Instana headers have priority)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I need to wait for an internal confirmation if Instana header has less priority or not.

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@aryamohanan aryamohanan left a comment

Choose a reason for hiding this comment

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

LGTM

@kirrg001
Copy link
Copy Markdown
Contributor Author

Waiting for #2512 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants