-
Notifications
You must be signed in to change notification settings - Fork 144
Open
Open
Copy link
Labels
Issue type - bugBugs in existing code that needs to be fixed.Bugs in existing code that needs to be fixed.Plugin - local_o365Status - PR ready / pending releaseDev is done and PR ready. Will be included in the next release.Dev is done and PR ready. Will be included in the next release.
Milestone
Description
When using the PowerShell script to create new Azure app for the integration, if the Moodle URL isn't from a tenant verified domain, the script run will throw the attached error message when trying to add identifier URI. This will not result in the script running fail - the app would still be created with correct redirect URIs, permissions and admin consents etc, but all settings in the "Expose an API" configuration page will be missing.

To solution to this is to check if the Moodle URL is from a tenant verified domain, and only try to use it to form the identifier URI if it contains it; otherwise use the default identifier URI without Moodle URL.
Metadata
Metadata
Assignees
Labels
Issue type - bugBugs in existing code that needs to be fixed.Bugs in existing code that needs to be fixed.Plugin - local_o365Status - PR ready / pending releaseDev is done and PR ready. Will be included in the next release.Dev is done and PR ready. Will be included in the next release.