Skip to content

Respond to review comment about WithExtraQueryParameters obsolescence tracking#3703

Closed
Copilot wants to merge 1 commit intomarcinzo/idweb_custom_claimsfrom
copilot/sub-pr-3623
Closed

Respond to review comment about WithExtraQueryParameters obsolescence tracking#3703
Copilot wants to merge 1 commit intomarcinzo/idweb_custom_claimsfrom
copilot/sub-pr-3623

Conversation

Copy link
Contributor

Copilot AI commented Feb 4, 2026

Addressed review feedback on PR #3623 requesting creation of a GitHub issue to track updating WithExtraQueryParameters to the latest MSAL API.

Changes

  • Responded to review comment explaining I cannot create GitHub issues directly
  • Documented 5 locations in TokenAcquisition.cs where WithExtraQueryParameters is used with obsolete pragma warnings (lines 159, 439, 638, 1207, 1371)
  • Requested manual issue creation by reviewer

Context

MSAL has deprecated WithExtraQueryParameters. The codebase currently suppresses obsolete warnings with #pragma warning disable CS0618. This needs to be tracked for future migration to the newer MSAL overload.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] WIP Address feedback on WithExtraQueryParameters in token acquisition Respond to review comment about WithExtraQueryParameters obsolescence tracking Feb 4, 2026
Copilot AI requested a review from trwalke February 4, 2026 00:22
@trwalke trwalke closed this Feb 4, 2026
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.

2 participants