Skip to content

Commit c73b7ca

Browse files
iulico-1rayluo
authored andcommitted
update the default broker redirect uri
Ray: I tested it on his Win laptop to successfully acquire normal token and ssh cert from broker
1 parent 7e04519 commit c73b7ca

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

msal/broker.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -169,8 +169,9 @@ def _signin_interactively(
169169
**kwargs):
170170
params = pymsalruntime.MSALRuntimeAuthParameters(client_id, authority)
171171
params.set_requested_scopes(scopes)
172-
params.set_redirect_uri("placeholder") # pymsalruntime 0.1 requires non-empty str,
173-
# the actual redirect_uri will be overridden by a value hardcoded by the broker
172+
params.set_redirect_uri("https://login.microsoftonline.com/common/oauth2/nativeclient")
173+
# This default redirect_uri value is not currently used by the broker
174+
# but it is required by the MSAL.cpp to be set to a non-empty valid URI.
174175
if prompt:
175176
if prompt == "select_account":
176177
if login_hint:

0 commit comments

Comments
 (0)