Skip to content

Commit 27d165c

Browse files
committed
Do not prompt with no ARN set for device.
1 parent 0164f55 commit 27d165c

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

s3/src/main/java/ch/cyberduck/core/sts/STSAuthorizationService.java

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -129,17 +129,19 @@ public TemporaryAccessTokens getSessionToken(final Credentials credentials) thro
129129
}
130130
}
131131
}
132-
log.debug("Prompt for MFA token code");
133-
final String tokenCode = prompt.prompt(
134-
bookmark, String.format("%s %s", LocaleFactory.localizedString("Multi-Factor Authentication", "S3"),
135-
mfaArn),
136-
LocaleFactory.localizedString("Provide additional login credentials", "Credentials"),
137-
new LoginOptions(bookmark.getProtocol())
138-
.password(true)
139-
.passwordPlaceholder(LocaleFactory.localizedString("MFA Authentication Code", "S3"))
140-
.keychain(false)
141-
).getPassword();
142-
request.setTokenCode(tokenCode);
132+
if(request.getSerialNumber() != null) {
133+
log.debug("Prompt for MFA token code");
134+
final String tokenCode = prompt.prompt(
135+
bookmark, String.format("%s %s", LocaleFactory.localizedString("Multi-Factor Authentication", "S3"),
136+
mfaArn),
137+
LocaleFactory.localizedString("Provide additional login credentials", "Credentials"),
138+
new LoginOptions(bookmark.getProtocol())
139+
.password(true)
140+
.passwordPlaceholder(LocaleFactory.localizedString("MFA Authentication Code", "S3"))
141+
.keychain(false)
142+
).getPassword();
143+
request.setTokenCode(tokenCode);
144+
}
143145
log.debug("Request {} from {}", request, service);
144146
try {
145147
final GetSessionTokenResult result = service.getSessionToken(request);

0 commit comments

Comments
 (0)