fix: Use provided national cloud as base url

microsoftgraph · Dec 16, 2024 · 4323b09 · 4323b09
1 parent 80f8ab1
commit 4323b09
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/GraphServiceClient.php b/src/GraphServiceClient.php
@@ -45,9 +45,11 @@ public function __construct(
             parent::__construct($requestAdapter);
             return;
         }
-        parent::__construct(new GraphRequestAdapter(new GraphPhpLeagueAuthenticationProvider(
+        $defaultRequestAdapter = new GraphRequestAdapter(new GraphPhpLeagueAuthenticationProvider(
             $tokenRequestContext, $scopes, $nationalCloud
-        )));
+        ));
+        $defaultRequestAdapter->setBaseUrl($nationalCloud.'/v1.0');
+        parent::__construct($defaultRequestAdapter);
     }
 
     /**

diff --git a/tests/GraphServiceClientTest.php b/tests/GraphServiceClientTest.php
@@ -126,4 +126,10 @@ function (RequestInterface $request) {
         $this->assertInstanceOf(AccessToken::class, $newCache->getTokenWithContext($newTokenRequestContext));
         $this->assertEquals($this->testJWT, $newCache->getTokenWithContext($newTokenRequestContext)->getToken());
     }
+
+    public function testNationalCloudUsed(): void
+    {
+        $client = new GraphServiceClient(new ClientCredentialContext('tenant', 'client', 'secret'), [], NationalCloud::US_GOV);
+        $this->assertEquals(NationalCloud::US_GOV.'/v1.0', $client->getRequestAdapter()->getBaseUrl());
+    }
 }