mirror of
https://github.com/chatwoot/chatwoot.git
synced 2026-06-04 21:02:35 +08:00
b791d75b30
Fixes #9775 ## Description This fixes a repeated admin consent loop in the Microsoft OAuth flow when connecting a Microsoft email inbox. Chatwoot was always sending `prompt=consent` in the Microsoft authorization URL. In the current code path, this parameter is only used when building the authorization URL and is not required by the callback, token exchange, token persistence, or refresh flow. By removing the forced consent prompt, the OAuth flow can proceed normally without repeatedly sending users back through the admin consent screen. ## What changed - removed `prompt: 'consent'` from the Microsoft authorization URL - added a regression assertion to ensure `prompt` is not included in the generated URL ## Why this is safe - `redirect_uri`, `scope`, and `state` remain unchanged - callback and token exchange flow remain unchanged - refresh token flow remains unchanged - no other part of the current Microsoft inbox flow depends on forcing a consent screen ## Testing - updated controller spec to assert that the generated authorization URL does not include `prompt` |
||
|---|---|---|
| .. | ||
| api | ||
| auth | ||
| concerns | ||
| devise | ||
| devise_overrides | ||
| installation | ||
| linear | ||
| microsoft | ||
| notion | ||
| platform/api/v1 | ||
| public/api/v1 | ||
| service | ||
| shopify | ||
| super_admin | ||
| tiktok | ||
| twilio | ||
| webhooks | ||
| .keep | ||
| android_assetlinks_spec.rb | ||
| api_controller_spec.rb | ||
| apple_app_spec.rb | ||
| dashboard_controller_spec.rb | ||
| health_controller_spec.rb | ||
| microsoft_controller_spec.rb | ||
| slack_uploads_controller_spec.rb | ||
| swagger_controller_spec.rb | ||
| widget_tests_controller_spec.rb | ||
| widgets_controller_spec.rb | ||