feat: custom http client

[adrienbrault]

While microsoft/microsoft-graph 2.0-RC13 allows using a custom guzzle client, the OAuth authentication does not.

In order to use a custom client this is what's currently required:

$accessToken = $this->getAccessTokenUsingMyGuzzleClient();

$authenticationProvider = new BaseBearerTokenAuthenticationProvider(
    new class($accessToken) implements AccessTokenProvider {
        public function __construct(private AccessTokenInterface $accessToken)
        {
        }

        public function getAuthorizationTokenAsync(string $url): Promise
        {
            return new FulfilledPromise($this->accessToken);
        }

        public function getAllowedHostsValidator(): AllowedHostsValidator
        {
            return new AllowedHostsValidator([]);
        }
    }
);

return new GraphServiceClient(
    new GraphRequestAdapter(
        $authenticationProvider,
        $this->guzzleClient
    )
);

This patch should make it simpler:

$authenticationProvider = new BaseBearerTokenAuthenticationProvider(
    new PhpLeagueAccessTokenProvider(
        $this->tokenRequestContext,
        [],
        [],
        ProviderFactory::create(
            $this->tokenRequestContext,
            [
                'httpClient' => $this->guzzleClient,
            ]
        )
    )
);

return new GraphServiceClient(
    new GraphRequestAdapter(
        $authenticationProvider,
        $this->guzzleClient
    )
);

[Ndiritu]

Also @baywet how should we approach custom HTTP clients for authentication? Should we support passing down a custom HTTP client from the request adapter into the auth layer?

Please lmk if creating an issue on the main Kiota repo would be better for this discussion

[baywet]

Also @baywet how should we approach custom HTTP clients for authentication? Should we support passing down a custom HTTP client from the request adapter into the auth layer?

Please lmk if creating an issue on the main Kiota repo would be better for this discussion

Let's wait to see if there's any demand for that before we start putting the work.

[baywet]

@adrienbrault ça fait plaisir de croiser un ancien SUPINFO :)

[adrienbrault]

@baywet 👋 :-)

[Ndiritu]

@baywet unable to confirm if the SONAR_TOKEN secret is missing from this repo's settings. Kindly assist