Skip to content

Commit

Permalink
Use broker for SSH Cert feature
Browse files Browse the repository at this point in the history
It is OK to use PyMsalRuntime now, since ESTS has deployed the fix.
https://identitydivision.visualstudio.com/Engineering/_workitems/edit/2060332
  • Loading branch information
rayluo committed Nov 18, 2022
1 parent 42fc402 commit d1320d8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions msal/application.py
Original file line number Diff line number Diff line change
Expand Up @@ -1375,7 +1375,7 @@ def _acquire_token_silent_from_cache_and_possibly_refresh_it(
if account and account.get("authority_type") == _AUTHORITY_TYPE_CLOUDSHELL:
return self._acquire_token_by_cloud_shell(scopes, data=data)

if self._enable_broker and account is not None and data.get("token_type") != "ssh-cert":
if self._enable_broker and account is not None:
from .broker import _acquire_token_silently
response = _acquire_token_silently(
"https://{}/{}".format(self.authority.instance, self.authority.tenant),
Expand Down Expand Up @@ -1799,7 +1799,7 @@ def acquire_token_interactive(
return self._acquire_token_by_cloud_shell(scopes, data=data)
claims = _merge_claims_challenge_and_capabilities(
self._client_capabilities, claims_challenge)
if self._enable_broker and data.get("token_type") != "ssh-cert":
if self._enable_broker:
if parent_window_handle is None:
raise ValueError(
"parent_window_handle is required when you opted into using broker. "
Expand Down

0 comments on commit d1320d8

Please sign in to comment.