AzureAD / microsoft-authentication-library-for-js

Microsoft Authentication Library (MSAL) for JS
http://aka.ms/aadv2
MIT License
3.64k stars 2.65k forks source link

In the msal-node device code sample, increase the timeout value from 5 to prevent the login from timing out by default #3017

Closed waldekmastykarz closed 3 years ago

waldekmastykarz commented 3 years ago

Library

Documentation location

Description

In the msal-node device code auth sample, increase the timeout value from 5 to something else or consider removing it altogether. With timeout set to 5, the login times out every time while you're going navigating to the device login page and pasting the device code.

Additionally, it would be good to specify why this value is needed as the device code response sent initially from AAD already has a timeout of 900 specified. Is the timeout in the deviceCodeRequest meant to override it? If so, what's the purpose? It would be helpful to clarify this.

hectormmg commented 3 years ago

Hi @waldekmastykarz thanks for the feedback! We're in the process of polishing our samples and should have things like this updated soon. Some of these details are defaults that we use for testing purposes that we need to change for the usage samples.

hectormmg commented 3 years ago

Closing since this was addressed in #3031.