Skip to content

Drop project-id from threadcontext for CCS#136664

Merged
tvernum merged 2 commits intoelastic:mainfrom
tvernum:rcs/drop-project-id
Oct 16, 2025
Merged

Drop project-id from threadcontext for CCS#136664
tvernum merged 2 commits intoelastic:mainfrom
tvernum:rcs/drop-project-id

Conversation

@tvernum
Copy link
Contributor

@tvernum tvernum commented Oct 16, 2025

When making a request to a remote cluster, we don't want to include the current project-id ("X-Elastic-Project-Id") header in the thread context that we send to the remote cluster because the current project-id is not relevant to the thread context for the remote execution.

When making a request to a remote cluster, we don't want to include
the current project-id ("X-Elastic-Project-Id") header in the thread
context that we send to the remote cluster because the current
project-id is not relevant to the thread context for the remote
execution.
@tvernum tvernum requested review from a team and ywangd October 16, 2025 06:57
@tvernum tvernum requested a review from a team as a code owner October 16, 2025 06:57
@tvernum tvernum added >bug :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC v9.2.0 v9.1.6 labels Oct 16, 2025
@elasticsearchmachine elasticsearchmachine added Team:Security Meta label for security team v9.3.0 labels Oct 16, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

@elasticsearchmachine
Copy link
Collaborator

Hi @tvernum, I've created a changelog YAML for you.

@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Oct 16, 2025
@tvernum tvernum added v9.2.1 auto-backport Automatically create backport pull requests when merged and removed v9.2.0 labels Oct 16, 2025
Copy link
Contributor

@slobodanadamovic slobodanadamovic left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@tvernum tvernum merged commit 20e185d into elastic:main Oct 16, 2025
41 checks passed
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Oct 16, 2025

💔 Backport failed

Status Branch Result
9.1 Commit could not be cherrypicked due to conflicts
9.2 Commit could not be cherrypicked due to conflicts

9.2

9.1

tvernum added a commit to tvernum/elasticsearch that referenced this pull request Oct 16, 2025
When making a request to a remote cluster, we don't want to include
the current project-id ("X-Elastic-Project-Id") header in the thread
context that we send to the remote cluster because the current
project-id is not relevant to the thread context for the remote
execution.

Backport of: elastic#136664
elasticsearchmachine pushed a commit that referenced this pull request Oct 16, 2025
When making a request to a remote cluster, we don't want to include
the current project-id ("X-Elastic-Project-Id") header in the thread
context that we send to the remote cluster because the current
project-id is not relevant to the thread context for the remote
execution.

Backport of: #136664

Co-authored-by: Johannes Fredén <109296772+jfreden@users.noreply.github.com>
Kubik42 pushed a commit to Kubik42/elasticsearch that referenced this pull request Oct 16, 2025
When making a request to a remote cluster, we don't want to include
the current project-id ("X-Elastic-Project-Id") header in the thread
context that we send to the remote cluster because the current
project-id is not relevant to the thread context for the remote
execution.
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Oct 16, 2025
When making a request to a remote cluster, we don't want to include
the current project-id ("X-Elastic-Project-Id") header in the thread
context that we send to the remote cluster because the current
project-id is not relevant to the thread context for the remote
execution.

Backport of: elastic#136664,elastic#136675
elasticsearchmachine pushed a commit that referenced this pull request Oct 17, 2025
When making a request to a remote cluster, we don't want to include
the current project-id ("X-Elastic-Project-Id") header in the thread
context that we send to the remote cluster because the current
project-id is not relevant to the thread context for the remote
execution.

Backport of: #136664,#136675
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged >bug :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC serverless-linked Added by automation, don't add manually Team:Security Meta label for security team v9.1.6 v9.2.1 v9.3.0

3 participants