Skip to content

[backport][Ti_AbuseCH] - Added support for optional "Auth Key" request header across all data streams - #13317

Merged
ShourieG merged 6 commits into
elastic:backport-ti_abusech-2.6from
ShourieG:ti_abusech/bugfix-2.6.0
Mar 27, 2025
Merged

[backport][Ti_AbuseCH] - Added support for optional "Auth Key" request header across all data streams #13317
ShourieG merged 6 commits into
elastic:backport-ti_abusech-2.6from
ShourieG:ti_abusech/bugfix-2.6.0

Conversation

@ShourieG

@ShourieG ShourieG commented Mar 26, 2025

Copy link
Copy Markdown
Contributor

Type of change

  • bug

Proposed commit message

Added support for optional "Auth Key" request header across all data streams. This PR is done as a backport bugfix to version 2.6.0 and this PR, so that the feature is available to users with elastic stack 8.13.0 and above without having to upgrade to 8.18.0.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

Screenshot 2025-03-21 at 10 43 28 PM
@ShourieG
ShourieG requested a review from a team as a code owner March 26, 2025 14:28
@ShourieG ShourieG self-assigned this Mar 26, 2025
@ShourieG ShourieG added Integration:ti_abusech abuse.ch Team:Service-Integrations Label for the Observability Service Integrations team bugfix Pull request that fixes a bug issue enhancement New feature or request backport labels Mar 26, 2025
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@andrewkroh andrewkroh added the Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] label Mar 26, 2025
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@ShourieG
ShourieG requested review from efd6 and removed request for efd6 March 27, 2025 01:20

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Screenshot 2025-03-27 at 3 04 31 PM

@ShourieG, not sure why your branch is out of sync with backport branch. Was there any changes to backport branch?

@ShourieG

ShourieG commented Mar 27, 2025

Copy link
Copy Markdown
Contributor Author

@kcreddy, so there was a commit made by vault-plugin bot in the backport branch -

commit 9851206e7c83019c6033ff7ee84d4c663d964969 (HEAD, upstream/backport-ti_abusech-2.6)
Author: elastic-vault-github-plugin-prod <elasticmachine@elastic.co>
Date:   Wed Mar 26 12:57:45 2025 +0000

    Add .buildkite and .ci to backport branch: backport-ti_abusech-2.6 from the main branch

This is just adding configs and ci files so this should be ok.

@kcreddy

kcreddy commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

@ShourieG , I think we should remove this label: Team:Service-Integrations, looks like its for Observability.

@ShourieG

Copy link
Copy Markdown
Contributor Author

@ShourieG , I think we should remove this label: Team:Service-Integrations, looks like its for Observability.

yea, added by mistake

@ShourieG ShourieG removed the Team:Service-Integrations Label for the Observability Service Integrations team label Mar 27, 2025
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @ShourieG

@ShourieG
ShourieG merged commit b83d356 into elastic:backport-ti_abusech-2.6 Mar 27, 2025
@ShourieG
ShourieG deleted the ti_abusech/bugfix-2.6.0 branch March 27, 2025 11:02
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package ti_abusech - 2.6.1 containing this change is available at https://epr.elastic.co/package/ti_abusech/2.6.1/

@kevinlincdf

kevinlincdf commented Mar 28, 2025

Copy link
Copy Markdown

Thank you all.
The rate-limit problem was solved.

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

Labels

backport bugfix Pull request that fixes a bug issue enhancement New feature or request Integration:ti_abusech abuse.ch Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

5 participants