Skip to content

Commit a66aecf

Browse files
committed
S2A utility returns S2AConfig.
1 parent 7c5ed2f commit a66aecf

File tree

3 files changed

+20
-20
lines changed

3 files changed

+20
-20
lines changed

oauth2_http/java/com/google/auth/oauth2/S2A.java

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@
5858
* <p>This is an experimental utility.
5959
*/
6060
@ThreadSafe
61-
public final class S2A {
61+
public class S2A {
6262
static final String S2A_PLAINTEXT_ADDRESS_JSON_KEY = "plaintext_address";
6363
static final String S2A_MTLS_ADDRESS_JSON_KEY = "mtls_address";
6464
static final String S2A_CONFIG_ENDPOINT_POSTFIX =
@@ -81,14 +81,9 @@ public final class S2A {
8181
this.config = getS2AConfigFromMDS();
8282
}
8383

84-
/** @return the mTLS S2A Address from the mTLS config. */
85-
public String getMtlsS2AAddress() {
86-
return config.getMtlsAddress();
87-
}
88-
89-
/** @return the plaintext S2A Address from the mTLS config. */
90-
public String getPlaintextS2AAddress() {
91-
return config.getPlaintextAddress();
84+
/** @return the cached S2AConfig. */
85+
public S2AConfig getConfigFromMDS() {
86+
return config;
9287
}
9388

9489
public static Builder newBuilder() {

oauth2_http/java/com/google/auth/oauth2/S2AConfig.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
import com.google.errorprone.annotations.CanIgnoreReturnValue;
3434

3535
/** Holds an mTLS configuration (consists of address of S2A) retrieved from the Metadata Server. */
36-
final class S2AConfig {
36+
public class S2AConfig {
3737
// plaintextAddress is the plaintext address to reach the S2A.
3838
private final String plaintextAddress;
3939

oauth2_http/javatests/com/google/auth/oauth2/S2ATest.java

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,9 @@ public void getS2AAddress_validAddress() {
6060
transportFactory.transport.setRequestStatusCode(HttpStatusCodes.STATUS_CODE_OK);
6161

6262
S2A s2aUtils = S2A.newBuilder().setHttpTransportFactory(transportFactory).build();
63-
String plaintextS2AAddress = s2aUtils.getPlaintextS2AAddress();
64-
String mtlsS2AAddress = s2aUtils.getMtlsS2AAddress();
63+
S2AConfig config = s2aUtils.getConfigFromMDS();
64+
String plaintextS2AAddress = config.getPlaintextAddress();
65+
String mtlsS2AAddress = config.getMtlsAddress();
6566
assertEquals(S2A_PLAINTEXT_ADDRESS, plaintextS2AAddress);
6667
assertEquals(S2A_MTLS_ADDRESS, mtlsS2AAddress);
6768
}
@@ -79,8 +80,9 @@ public void getS2AAddress_queryEndpointResponseErrorCode_emptyAddress() {
7980
HttpStatusCodes.STATUS_CODE_SERVICE_UNAVAILABLE);
8081

8182
S2A s2aUtils = S2A.newBuilder().setHttpTransportFactory(transportFactory).build();
82-
String plaintextS2AAddress = s2aUtils.getPlaintextS2AAddress();
83-
String mtlsS2AAddress = s2aUtils.getMtlsS2AAddress();
83+
S2AConfig config = s2aUtils.getConfigFromMDS();
84+
String plaintextS2AAddress = config.getPlaintextAddress();
85+
String mtlsS2AAddress = config.getMtlsAddress();
8486
assertTrue(plaintextS2AAddress.isEmpty());
8587
assertTrue(mtlsS2AAddress.isEmpty());
8688
}
@@ -98,8 +100,9 @@ public void getS2AAddress_queryEndpointResponseEmpty_emptyAddress() {
98100
transportFactory.transport.setEmptyContent(true);
99101

100102
S2A s2aUtils = S2A.newBuilder().setHttpTransportFactory(transportFactory).build();
101-
String plaintextS2AAddress = s2aUtils.getPlaintextS2AAddress();
102-
String mtlsS2AAddress = s2aUtils.getMtlsS2AAddress();
103+
S2AConfig config = s2aUtils.getConfigFromMDS();
104+
String plaintextS2AAddress = config.getPlaintextAddress();
105+
String mtlsS2AAddress = config.getMtlsAddress();
103106
assertTrue(plaintextS2AAddress.isEmpty());
104107
assertTrue(mtlsS2AAddress.isEmpty());
105108
}
@@ -116,8 +119,9 @@ public void getS2AAddress_queryEndpointResponseInvalidPlaintextJsonKey_plaintext
116119
transportFactory.transport.setRequestStatusCode(HttpStatusCodes.STATUS_CODE_OK);
117120

118121
S2A s2aUtils = S2A.newBuilder().setHttpTransportFactory(transportFactory).build();
119-
String plaintextS2AAddress = s2aUtils.getPlaintextS2AAddress();
120-
String mtlsS2AAddress = s2aUtils.getMtlsS2AAddress();
122+
S2AConfig config = s2aUtils.getConfigFromMDS();
123+
String plaintextS2AAddress = config.getPlaintextAddress();
124+
String mtlsS2AAddress = config.getMtlsAddress();
121125
assertTrue(plaintextS2AAddress.isEmpty());
122126
assertEquals(S2A_MTLS_ADDRESS, mtlsS2AAddress);
123127
}
@@ -134,8 +138,9 @@ public void getS2AAddress_queryEndpointResponseInvalidMtlsJsonKey_mtlsEmptyAddre
134138
transportFactory.transport.setRequestStatusCode(HttpStatusCodes.STATUS_CODE_OK);
135139

136140
S2A s2aUtils = S2A.newBuilder().setHttpTransportFactory(transportFactory).build();
137-
String plaintextS2AAddress = s2aUtils.getPlaintextS2AAddress();
138-
String mtlsS2AAddress = s2aUtils.getMtlsS2AAddress();
141+
S2AConfig config = s2aUtils.getConfigFromMDS();
142+
String plaintextS2AAddress = config.getPlaintextAddress();
143+
String mtlsS2AAddress = config.getMtlsAddress();
139144
assertEquals(S2A_PLAINTEXT_ADDRESS, plaintextS2AAddress);
140145
assertTrue(mtlsS2AAddress.isEmpty());
141146
}

0 commit comments

Comments
 (0)