Skip to content

chore(appsec): tag spans with refreshed rc client id - #19819

Draft
litianningdatadog wants to merge 1 commit into
tianning.li/3-1-remoteconfig-identity-clientfrom
tianning.li/3-2-appsec-rc-client-id
Draft

chore(appsec): tag spans with refreshed rc client id#19819
litianningdatadog wants to merge 1 commit into
tianning.li/3-1-remoteconfig-identity-clientfrom
tianning.li/3-2-appsec-rc-client-id

Conversation

@litianningdatadog

Copy link
Copy Markdown
Contributor

Description

Split from #19780.

AppSec reports the Remote Config client id on spans. Read the live Remote Config client id when tagging spans so AppSec reflects a refreshed client id after MicroVM runtime identity refresh.

Testing

  • scripts/lint format_check ddtrace/appsec/_asm_request_context.py ddtrace/appsec/_remoteconfiguration.py ddtrace/internal/settings/asm.py tests/appsec/appsec/test_asm_request_context.py tests/appsec/appsec/test_remoteconfiguration.py
  • git diff --check

Stack

Draft split branch. Stacked on the Remote Config split PR.

@litianningdatadog litianningdatadog added changelog/no-changelog A changelog entry is not required for this PR. aws-microvm Work related to AWS MicroVM onboarding labels Aug 23, 2026
@litianningdatadog litianningdatadog changed the title fix(appsec): tag spans with refreshed rc client id chore(appsec): tag spans with refreshed rc client id Aug 23, 2026
@datadog-prod-us1-4

This comment has been minimized.

@litianningdatadog
litianningdatadog force-pushed the tianning.li/3-1-remoteconfig-identity-client branch from 3e763c0 to ff3e857 Compare August 24, 2026 02:24
@litianningdatadog
litianningdatadog force-pushed the tianning.li/3-2-appsec-rc-client-id branch from 4d1e2f9 to f070e18 Compare August 24, 2026 02:24
@litianningdatadog
litianningdatadog force-pushed the tianning.li/3-1-remoteconfig-identity-client branch from ff3e857 to 25a94b2 Compare August 24, 2026 02:31
@litianningdatadog
litianningdatadog force-pushed the tianning.li/3-2-appsec-rc-client-id branch from f070e18 to eda6c45 Compare August 24, 2026 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aws-microvm Work related to AWS MicroVM onboarding changelog/no-changelog A changelog entry is not required for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant