Skip to content

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

Open
DimaBir wants to merge 4 commits intomainfrom
feature/add-subscription-tenant-telemetry
Open

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

Conversation

@DimaBir
Copy link
Contributor

@DimaBir DimaBir commented Feb 17, 2026

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

@DimaBir DimaBir self-assigned this Feb 17, 2026
@DimaBir DimaBir requested a review from jbrotsos February 17, 2026 20:02
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