Use with passport-azure-ad about passport-oauth2-refresh

[SigurdMW]

I'm also using passport-azure-ad and hoping to use this package.

[DayS]

The same issue is present with passport-openidconnect package, due to the exact same check.

For this one, the oauth2 instance is retrieved just before using it via a method defined in this strategy (not by using inheritance)

So, instead of always relying on a field which may no be present, maybe we should just let the possibility for the user to provide the Oauth2 configuration ?

[fiznool]

I'm very open to PRs that fix this behaviour!

[fiznool]

As of v2.1.0, there's a new configuration option setRefreshOAuth2 which allows you to use a custom OAuth2 adapter for the refresh token, which should hopefully solve this issue. For passport-azure-ad, assuming you are using the OIDC strategy, you'll need to replicate the configuration as shown in the createOAuth2Instance function.

Please give it a try and let me know if you have any feedback.

[alexbran8]

can anyone please share an example with passport-azure-ad? I have two issues which I cannot solve (1. strategy was not registered to refresh a token; 2. the callback from passport-azure-ad, does not add both refresh and access tokens on initial login...)