Skip to content

Commit

Permalink
update the default broker redirect uri
Browse files Browse the repository at this point in the history
Ray: I tested it on his Win laptop to successfully acquire normal token and ssh cert from broker
  • Loading branch information
iulico-1 authored and rayluo committed Mar 5, 2024
1 parent 7e04519 commit c73b7ca
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions msal/broker.py
Original file line number Diff line number Diff line change
Expand Up @@ -169,8 +169,9 @@ def _signin_interactively(
**kwargs):
params = pymsalruntime.MSALRuntimeAuthParameters(client_id, authority)
params.set_requested_scopes(scopes)
params.set_redirect_uri("placeholder") # pymsalruntime 0.1 requires non-empty str,
# the actual redirect_uri will be overridden by a value hardcoded by the broker
params.set_redirect_uri("https://login.microsoftonline.com/common/oauth2/nativeclient")
# This default redirect_uri value is not currently used by the broker
# but it is required by the MSAL.cpp to be set to a non-empty valid URI.
if prompt:
if prompt == "select_account":
if login_hint:
Expand Down

0 comments on commit c73b7ca

Please sign in to comment.