Skip to content

Commit 9271d65

Browse files
authored
Merge pull request #21880 from nextcloud/webauthn/discourage_user_auth
Discourage webauthn user interaction
2 parents 5add0ff + 884f3d0 commit 9271d65

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

lib/private/Authentication/WebAuthn/Manager.php

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,11 @@ public function startRegistration(IUser $user, string $serverHost): PublicKeyCre
107107
$excludedPublicKeyDescriptors = [
108108
];
109109

110-
$authenticatorSelectionCriteria = new AuthenticatorSelectionCriteria();
110+
$authenticatorSelectionCriteria = new AuthenticatorSelectionCriteria(
111+
null,
112+
false,
113+
AuthenticatorSelectionCriteria::USER_VERIFICATION_REQUIREMENT_DISCOURAGED
114+
);
111115

112116
return new PublicKeyCredentialCreationOptions(
113117
$rpEntity,
@@ -186,7 +190,8 @@ public function startAuthentication(string $uid, string $serverHost): PublicKeyC
186190
random_bytes(32), // Challenge
187191
60000, // Timeout
188192
$this->stripPort($serverHost), // Relying Party ID
189-
$registeredPublicKeyCredentialDescriptors // Registered PublicKeyCredentialDescriptor classes
193+
$registeredPublicKeyCredentialDescriptors, // Registered PublicKeyCredentialDescriptor classes
194+
AuthenticatorSelectionCriteria::USER_VERIFICATION_REQUIREMENT_DISCOURAGED
190195
);
191196
}
192197

0 commit comments

Comments
 (0)