Skip to content

Commit 45679a1

Browse files
Fix lint
1 parent 2b9d03d commit 45679a1

File tree

2 files changed

+20
-11
lines changed

2 files changed

+20
-11
lines changed

src/main/java/net/snowflake/client/core/auth/wif/AzureIdentityAttestationCreator.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,12 @@
44
import static net.snowflake.client.core.auth.wif.WorkloadIdentityUtil.SubjectAndIssuer;
55
import static net.snowflake.client.core.auth.wif.WorkloadIdentityUtil.extractClaimsWithoutVerifyingSignature;
66

7-
import java.util.Arrays;
8-
import java.util.HashSet;
9-
import java.util.Set;
10-
117
import com.fasterxml.jackson.databind.JsonNode;
128
import com.fasterxml.jackson.databind.ObjectMapper;
139
import com.google.common.base.Strings;
10+
import java.util.Arrays;
11+
import java.util.HashSet;
12+
import java.util.Set;
1413
import net.snowflake.client.core.SFLoginInput;
1514
import net.snowflake.client.core.SnowflakeJdbcInternalApi;
1615
import net.snowflake.client.log.SFLogger;
@@ -25,9 +24,8 @@ public class AzureIdentityAttestationCreator implements WorkloadIdentityAttestat
2524
public static final ObjectMapper objectMapper = new ObjectMapper();
2625

2726
private static final Set<String> ALLOWED_AZURE_TOKEN_ISSUER_PREFIXES =
28-
new HashSet<>(Arrays.asList(
29-
"https://sts.windows.net/",
30-
"https://login.microsoftonline.com/"));
27+
new HashSet<>(
28+
Arrays.asList("https://sts.windows.net/", "https://login.microsoftonline.com/"));
3129
private static final String DEFAULT_WORKLOAD_IDENTITY_ENTRA_RESOURCE =
3230
"api://fd3f753b-eed3-462c-b6a7-a4b5bb650aad";
3331

src/test/java/net/snowflake/client/core/auth/wif/AzureIdentityAttestationCreatorLatestIT.java

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,10 @@ public void successfulFlowV2IssuerScenario() {
123123
importMappingFromResources(SUCCESSFUL_FLOW_V2_ISSUER_SCENARIO_MAPPINGS);
124124
SFLoginInput loginInput = createLoginInputStub();
125125
AzureAttestationService attestationServiceMock = createAttestationServiceSpyForBasicFLow();
126-
executeAndAssertCorrectAttestationWithIssuer(attestationServiceMock, loginInput, "https://login.microsoftonline.com/fa15d692-e9c7-4460-a743-29f29522229/");
126+
executeAndAssertCorrectAttestationWithIssuer(
127+
attestationServiceMock,
128+
loginInput,
129+
"https://login.microsoftonline.com/fa15d692-e9c7-4460-a743-29f29522229/");
127130
}
128131

129132
@Test
@@ -151,7 +154,10 @@ public void successfulFlowAzureFunctionsWithV2IssuerScenario() {
151154
.thenReturn("some-identity-header-from-env");
152155
Mockito.when(attestationServiceSpy.getClientId()).thenReturn("managed-client-id-from-env");
153156

154-
executeAndAssertCorrectAttestationWithIssuer(attestationServiceSpy, loginInput, "https://login.microsoftonline.com/fa15d692-e9c7-4460-a743-29f29522229/");
157+
executeAndAssertCorrectAttestationWithIssuer(
158+
attestationServiceSpy,
159+
loginInput,
160+
"https://login.microsoftonline.com/fa15d692-e9c7-4460-a743-29f29522229/");
155161
}
156162

157163
@Test
@@ -244,11 +250,16 @@ private static AzureAttestationService createAttestationServiceSpyForBasicFLow()
244250

245251
private void executeAndAssertCorrectAttestation(
246252
AzureAttestationService attestationServiceMock, SFLoginInput loginInput) {
247-
executeAndAssertCorrectAttestationWithIssuer(attestationServiceMock, loginInput, "https://sts.windows.net/fa15d692-e9c7-4460-a743-29f29522229/");
253+
executeAndAssertCorrectAttestationWithIssuer(
254+
attestationServiceMock,
255+
loginInput,
256+
"https://sts.windows.net/fa15d692-e9c7-4460-a743-29f29522229/");
248257
}
249258

250259
private void executeAndAssertCorrectAttestationWithIssuer(
251-
AzureAttestationService attestationServiceMock, SFLoginInput loginInput, String expectedIssuer) {
260+
AzureAttestationService attestationServiceMock,
261+
SFLoginInput loginInput,
262+
String expectedIssuer) {
252263
AzureIdentityAttestationCreator attestationCreator =
253264
new AzureIdentityAttestationCreator(attestationServiceMock, loginInput, getBaseUrl());
254265

0 commit comments

Comments
 (0)