Skip to content

Commit ea8f81a

Browse files
bcthom09André Behrens
authored andcommitted
Fix: AutodiscoverService has no timeout, and thus might hang indefinitely.
1 parent 764f0dd commit ea8f81a

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

src/main/java/microsoft/exchange/webservices/data/autodiscover/AutodiscoverService.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -401,6 +401,7 @@ private URI getRedirectUrl(String domainName)
401401

402402
request.setRequestMethod("GET");
403403
request.setAllowAutoRedirect(false);
404+
request.setTimeout(getTimeout());
404405

405406
// Do NOT allow authentication as this single request will be made over plain HTTP.
406407
request.setAllowAuthentication(false);
@@ -1523,6 +1524,7 @@ private boolean tryGetEnabledEndpointsForHost(String host,
15231524
request.setAllowAutoRedirect(false);
15241525
request.setPreAuthenticate(false);
15251526
request.setUseDefaultCredentials(this.getUseDefaultCredentials());
1527+
request.setTimeout(getTimeout());
15261528

15271529
prepareCredentials(request);
15281530

src/main/java/microsoft/exchange/webservices/data/core/ExchangeService.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3607,7 +3607,8 @@ private URI getAutodiscoverUrl(String emailAddress,
36073607

36083608
AutodiscoverService autodiscoverService = new AutodiscoverService(this, requestedServerVersion);
36093609
autodiscoverService.setWebProxy(getWebProxy());
3610-
3610+
autodiscoverService.setTimeout(getTimeout());
3611+
36113612
autodiscoverService
36123613
.setRedirectionUrlValidationCallback(validateRedirectionUrlCallback);
36133614
autodiscoverService.setEnableScpLookup(this.getEnableScpLookup());

0 commit comments

Comments
 (0)