-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New authorization URI is a breaking change #2361
Copy link
Copy link
Closed
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Metadata
Metadata
Assignees
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
See #2275
Updating to the v2 auth URL broke the behavior of using
setApprovalPrompt('force'), which was deprecated in favor of instead of the newersetPrompt('consent').In order to mitigate this breaking change, we could update the
setApprovalPromptfunction to callsetPrompt('consent')instead.