Skip to content

Conversation

@tpavlovi
Copy link
Contributor

@tpavlovi tpavlovi commented Dec 18, 2025

Context

This PR implements late-bound idToken support in the AzureCLIV3 task, controlled by the EnableLateBoundIdToken feature flag.
AB#2341991


Task Name

AzureCLIV3


Description

Modified getIdToken() method to check for late-bound idToken when feature flag is enabled
The same change was added for the AzureCLIV2 task in this PR: #21580


Risk Assessment (Low / Medium / High)

Low


Change Behind Feature Flag (Yes / No)

Yes (EnableLateBoundIdToken)


Tech Design / Approach


Documentation Changes Required (Yes/No)

No


Unit Tests Added or Updated (Yes / No)

Yes


Additional Testing Performed


Logging Added/Updated (Yes/No)

Yes


Telemetry Added/Updated (Yes/No)

  • Custom telemetry (e.g., counters, timers, error tracking) is added as needed.
  • Events are tagged with proper metadata for filtering and analysis.
  • Telemetry is validated in staging or test environments.

Rollback Scenario and Process (Yes/No)

Revert dynamic FF EnableLateBoundIdToken


Dependency Impact Assessed and Regression Tested (Yes/No)

Yes


Checklist

  • Related issue linked (if applicable)
  • Task version was bumped — see versioning guide
  • Verified the task behaves as expected

@tpavlovi tpavlovi requested review from a team and manolerazvan as code owners December 18, 2025 15:38
@tpavlovi
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@tpavlovi tpavlovi force-pushed the users/tpavlovi/idToken branch from 093629a to 17e65c3 Compare December 18, 2025 16:07
@tpavlovi
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@tpavlovi tpavlovi force-pushed the users/tpavlovi/idToken branch from 17e65c3 to 3cc03ba Compare December 18, 2025 16:24
@tpavlovi
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

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.

3 participants