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.

Sign up for GitHub

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

Jump to bottom

add documentation and warning log for deprecating AccessTokenPassthrough #37575

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

add documentation and warning log for deprecating AccessTokenPassthrough #37575

wants to merge 3 commits into from
+53 −0

Conversation

asreehari-splunk
Copy link
Contributor

@asreehari-splunk asreehari-splunk commented Jan 29, 2025 •
edited by atoulme
Loading

Description

access_token_passthrough config is being deprecated in favor of enabling include_metadata in the receiver. As shown in the config and in the changelog also update batch processor as follows

  batch:
    metadata_keys: [X-Sf-Token]

Issue:

#37575

@asreehari-splunk asreehari-splunk requested review from dmitryax and a team as code owners January 29, 2025 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atoulme atoulme atoulme left review comments

@dmitryax dmitryax Awaiting requested review from dmitryax dmitryax is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@dmitryax dmitryax

Labels
receiver/signalfx
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@asreehari-splunk @atoulme @dmitryax