Skip to content

feat(telemetry): add SubscriptionId and TenantId passthrough for MSDO CLI - #159

Draft
Dima Birenbaum (DimaBir) wants to merge 4 commits into
mainfrom
feature/add-subscription-tenant-telemetry
Draft

feat(telemetry): add SubscriptionId and TenantId passthrough for MSDO CLI#159
Dima Birenbaum (DimaBir) wants to merge 4 commits into
mainfrom
feature/add-subscription-tenant-telemetry

Conversation

@DimaBir

Copy link
Copy Markdown
Collaborator

Summary

  • Add optional subscriptionId and tenantId inputs to the GitHub Action
  • Forward them as MSDO_SUBSCRIPTIONID / MSDO_TENANTID environment variables to the MSDO CLI process
  • Fully backward compatible — no behavior change when inputs are omitted

Context

Supports ADO PBI 36365209 — enabling MSDO CLI telemetry correlation with Defender for Cloud (dCSPM) subscription data.

Task: AB#36805549
Parent PBI: AB#36805486

Changes

File Change
action.yml +2 optional inputs: subscriptionId, tenantId
src/msdo.ts Set env vars before client.run() when inputs are provided
lib/msdo.js Compiled output

Test plan

  • Build passes (npm run build)
  • No regression — existing tests have same results as main
  • Env vars only set when inputs are provided (null-safe via isNullOrWhiteSpace)
  • Works for both run and upload code paths
  • E2E: validate CLI picks up env vars once CLI-side changes land

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant