Leaked Metadata

+++ b/src/Controller/MsLoginController.php
@@ -44,7 +44,7 @@ class MsLoginController extends ControllerBase {
   public function login() {
     $authorizationUrl = $this->azureProvider->getAuthorizationUrl([
-      'redirect_uri' => Url::fromRoute('phpmailer_oauth2.settings')->setAbsolute()->toString(),
+      'redirect_uri' => Url::fromRoute('phpmailer_oauth2.settings')->setAbsolute()->toString(TRUE),
       'scope' => $this->azureProvider->scope,
     ]); 

Is the return type the same? Or should it be:

'redirect_uri' => Url::fromRoute('phpmailer_oauth2.settings')->setAbsolute()->toString(TRUE)->getGeneratedUrl()

Url::toString(TRUE) returns a GeneratedUrl object, appending getGeneratedUrl() will return a string.

Which is what is returned without this patch.

With redirect_uri' => Url::fromRoute('phpmailer_oauth2.settings')->setAbsolute()->toString(TRUE)->getGeneratedUrl(), Azure returns an error because /admin/config/system/phpmailer-oauth2 is not the callback url defined in the Azure app.

How does adding "TRUE" fix the error? It's the same (incorrect?) route being returned, just as a GeneratedUrl object instead of a string.

Increasing the priority of this issue to "Major" because the module is currently unusable without a workaround.

I got the same error as above, but applying the patch allowed me to successfully generate the tokens:

Auth token retrieved.
Access token retrieved.
Refresh token retrieved.

I have no clue why returning a GeneratedUrl fixes the bug, though, hence the "Needs Review" status.

@frymorgan, perhaps you can speak more about this issue?

the module is currently unusable without a workaround.

I don't fully understand why as we've used this module in the past without it. That said, it works for at least 2 people so I've committed it.