Skip to content

Support b3multi propagation#1949

Merged
trask merged 1 commit into
vnextfrom
propagation-option
Mar 2, 2022
Merged

Support b3multi propagation#1949
trask merged 1 commit into
vnextfrom
propagation-option

Conversation

@trask

@trask trask commented Nov 4, 2021 •
edited
Loading

Copy link
Copy Markdown
Member

Resolves #1883

{
  "preview": {
    "additionalPropagators": [
      "b3multi"
    ]
  }
}

attaching CI snapshot for testing: applicationinsights-agent-3.2.3-BETA-SNAPSHOT.jar.zip

@trask trask marked this pull request as ready for review November 4, 2021 01:48
@trask trask mentioned this pull request Nov 4, 2021
Closed
@heyams

heyams commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

it will be better to include some tests?

@trask trask force-pushed the propagation-option branch from 67dd67a to 6643ae4 Compare February 16, 2022 21:04
@trask trask force-pushed the propagation-option branch from cb1434d to 13f59aa Compare February 17, 2022 00:49
@trask trask changed the base branch from main to vnext February 17, 2022 04:36
@trask trask closed this Feb 17, 2022
@trask trask reopened this Feb 17, 2022
@trask trask merged commit 3925803 into vnext Mar 2, 2022
@trask trask deleted the propagation-option branch March 2, 2022 17:15
@xsmrcek xsmrcek mentioned this pull request Mar 15, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kryalama kryalama Awaiting requested review from kryalama

@xiao-lix xiao-lix Awaiting requested review from xiao-lix

1 more reviewer

@heyams heyams heyams approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Double traceparent headers

2 participants

@trask @heyams