Skip to content

Commit 414686b

Browse files
Simplify
1 parent 3b06457 commit 414686b

File tree

3 files changed

+2
-6
lines changed

3 files changed

+2
-6
lines changed

databricks-sdk-java/src/main/java/com/databricks/sdk/core/oauth/Consent.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -113,10 +113,6 @@ public Builder withBrowserTimeout(Optional<Duration> browserTimeout) {
113113
return this;
114114
}
115115

116-
public Builder withBrowserTimeout(Duration browserTimeout) {
117-
return withBrowserTimeout(Optional.of(browserTimeout));
118-
}
119-
120116
public Consent build() {
121117
return new Consent(this);
122118
}

databricks-sdk-java/src/main/java/com/databricks/sdk/core/oauth/OAuthClient.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ public Consent initiateConsent() throws MalformedURLException {
211211
.withState(state)
212212
.withVerifier(verifier)
213213
.withHttpClient(hc)
214-
.withBrowserTimeout(browserTimeout.orElse(Duration.ofSeconds(0)))
214+
.withBrowserTimeout(browserTimeout)
215215
.build();
216216
}
217217
}

databricks-sdk-java/src/test/java/com/databricks/sdk/core/oauth/ConsentTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public void testConsentWithBrowserAuthTimeout() {
1919
.withRedirectUrl("http://localhost:8080/callback")
2020
.withState("test-state")
2121
.withVerifier("test-verifier")
22-
.withBrowserTimeout(Duration.ofSeconds(30))
22+
.withBrowserTimeout(Optional.of(Duration.ofSeconds(30)))
2323
.build();
2424

2525
// Verify that the timeout is properly set

0 commit comments

Comments
 (0)