[Identity] DeviceCodeCredential: Add a timeout optional parameter.

[sadasant]

Python currently offers a way to timeout the authentication request. We should investigate if we can add the same feature.

[sadasant]

Here's what I know so far,

DeviceCodeCredential (DCC) uses PublicClientApplication, from @azure/msal-node. PublicClientApplication extends ClientApplication, which has at least two distinct notions of a timeout:

• const hash = await interactionHandler.monitorPopupForHash(popupWindow, this.config.system.windowHashTimeout);
• const silentHandler = new SilentHandler(authClient, this.browserStorage, this.config.system.loadFrameTimeout);

I think that configuration makes sense to work with, but I might not have enough context.

[sadasant]

I had a conversation with @jonathandturner about this feature and how Python approaches it (here) and, since this issue relates to MSAL and to the work he's already doing, we decided on assigning this back to him! Thank you, Jonathan!

[sophiajt]

Request on MSAL for reference: AzureAD/microsoft-authentication-library-for-js#2494

[sadasant]

Turns out this is an inconsistency in Python. In JS it can be resolved with the abort signal. I'll close this for now.