Skip to content

Commit aa5e514

Browse files
committed
updated comments
1 parent c4a4b3b commit aa5e514

File tree

2 files changed

+1
-13
lines changed

2 files changed

+1
-13
lines changed

msal4j-sdk/src/main/java/com/microsoft/aad/msal4j/AadInstanceDiscoveryProvider.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ static InstanceDiscoveryMetadataEntry getMetadataEntry(URL authorityUrl,
8282
if(msalRequest.application().instanceDiscovery()){
8383
doInstanceDiscoveryAndCache(authorityUrl, validateAuthority, msalRequest, serviceBundle);
8484
} else {
85-
// instanceDiscovery flag is set to False. Do no perform instanceDiscovery.
85+
// instanceDiscovery flag is set to False. Do not perform instanceDiscovery.
8686
cache.putIfAbsent(host, InstanceDiscoveryMetadataEntry.builder().
8787
preferredCache(host).
8888
preferredNetwork(host).

msal4j-sdk/src/main/java/com/microsoft/aad/msal4j/AbstractClientApplicationBase.java

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -652,18 +652,6 @@ public T azureRegion(String val) {
652652
``https://login.microsoftonline.com`` to acquire some metadata, especially when using an unfamiliar authority.
653653
This behavior is known as Instance Discovery.
654654
This parameter defaults to true, which enables the Instance Discovery.
655-
If you know some authorities which you allow MSAL to operate with as-is,
656-
without involving any Instance Discovery, the recommended pattern is::
657-
knownAuthorities = frozenset([ # Treat your known authorities as const
658-
"https://contoso.com/adfs", "https://login.azs/foo"])
659-
...
660-
authority = "https://contoso.com/adfs" # Assuming your app will use this
661-
app1 = PublicClientApplication(
662-
"client_id",
663-
authority=authority,
664-
# Conditionally disable Instance Discovery for known authorities
665-
instance_discovery=authority not in known_authorities,
666-
)
667655
If you do not know some authorities beforehand,
668656
yet still want MSAL to accept any authority that you will provide,
669657
you can use a ``False`` to unconditionally disable Instance Discovery. */

0 commit comments

Comments
 (0)