Skip to content

Commit 8c647a8

Browse files
committed
Fix silent issue
1 parent 5ae3186 commit 8c647a8

File tree

2 files changed

+42
-7
lines changed

2 files changed

+42
-7
lines changed

msal4j-brokers/pom.xml

Lines changed: 40 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,11 @@
55
<modelVersion>4.0.0</modelVersion>
66
<groupId>com.microsoft.azure</groupId>
77
<artifactId>msal4j-brokers</artifactId>
8-
<version>1.0.0-beta</version>
8+
<version>1.0.2-beta</version>
99
<packaging>jar</packaging>
1010
<name>msal4j-brokers</name>
1111
<description>
12-
Microsoft Authentication Library for Java - Brokers helps you integrate with the broker
13-
on windows machine to secure Access tokens and refresh tokens.
12+
Microsoft Authentication Library for Java - Brokers is a companion package for MSAL Java that allows easy integration with authentication brokers
1413
</description>
1514
<url>https://github.com/AzureAD/microsoft-authentication-library-for-java</url>
1615
<licenses>
@@ -35,12 +34,12 @@
3534
<dependency>
3635
<groupId>com.microsoft.azure</groupId>
3736
<artifactId>msal4j</artifactId>
38-
<version>1.14.0-beta</version>
37+
<version>1.14.1-beta</version>
3938
</dependency>
4039
<dependency>
4140
<groupId>com.microsoft.azure</groupId>
4241
<artifactId>javamsalruntime</artifactId>
43-
<version>0.13.4</version>
42+
<version>0.13.8</version>
4443
</dependency>
4544
<dependency>
4645
<groupId>org.projectlombok</groupId>
@@ -65,6 +64,42 @@
6564
<version>1.2.3</version>
6665
<scope>test</scope>
6766
</dependency>
67+
<dependency>
68+
<groupId>commons-io</groupId>
69+
<artifactId>commons-io</artifactId>
70+
<version>2.11.0</version>
71+
<scope>test</scope>
72+
</dependency>
73+
<dependency>
74+
<groupId>org.seleniumhq.selenium</groupId>
75+
<artifactId>selenium-api</artifactId>
76+
<version>3.14.0</version>
77+
<scope>test</scope>
78+
</dependency>
79+
<dependency>
80+
<groupId>org.seleniumhq.selenium</groupId>
81+
<artifactId>selenium-chrome-driver</artifactId>
82+
<version>3.14.0</version>
83+
<scope>test</scope>
84+
</dependency>
85+
<dependency>
86+
<groupId>org.seleniumhq.selenium</groupId>
87+
<artifactId>selenium-support</artifactId>
88+
<version>3.14.0</version>
89+
<scope>test</scope>
90+
</dependency>
91+
<dependency>
92+
<groupId>com.azure</groupId>
93+
<artifactId>azure-core</artifactId>
94+
<version>1.22.0</version>
95+
<scope>test</scope>
96+
</dependency>
97+
<dependency>
98+
<groupId>com.azure</groupId>
99+
<artifactId>azure-security-keyvault-secrets</artifactId>
100+
<version>4.3.5</version>
101+
<scope>test</scope>
102+
</dependency>
68103
</dependencies>
69104

70105
<!-- force https -->

msal4j-brokers/src/main/java/com/microsoft/aad/msal4jbrokers/MsalRuntimeBroker.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public CompletableFuture<IAuthenticationResult> acquireToken(PublicClientApplica
4646
//If request has an account ID, MSALRuntime likely has data cached for that account that we can retrieve
4747
if (parameters.account() != null) {
4848
try {
49-
accountResult = ((ReadAccountResult) interop.readAccountById(parameters.account().homeAccountId(), application.correlationId()).get()).getAccount();
49+
accountResult = ((ReadAccountResult) interop.readAccountById(parameters.account().homeAccountId().split("\\.")[0], application.correlationId()).get()).getAccount();
5050
} catch (InterruptedException | ExecutionException ex) {
5151
throw new MsalClientException(String.format("MSALRuntime async operation interrupted when waiting for result: %s", ex.getMessage()), AuthenticationErrorCode.MSALRUNTIME_INTEROP_ERROR);
5252
}
@@ -142,7 +142,7 @@ public CompletableFuture<IAuthenticationResult> acquireToken(PublicClientApplica
142142
@Override
143143
public void removeAccount(PublicClientApplication application, IAccount msalJavaAccount) {
144144
try {
145-
Account msalRuntimeAccount = ((ReadAccountResult) interop.readAccountById(msalJavaAccount.homeAccountId(), application.correlationId()).get()).getAccount();
145+
Account msalRuntimeAccount = ((ReadAccountResult) interop.readAccountById(msalJavaAccount.homeAccountId().split("\\.")[0], application.correlationId()).get()).getAccount();
146146

147147
if (msalRuntimeAccount != null) {
148148
interop.signOutSilently(application.clientId(), application.correlationId(), msalRuntimeAccount);

0 commit comments

Comments
 (0)