Skip to content

Commit

Permalink
Merge pull request #16 from nevissecurity/feature/NEVISACCESSAPP-5886-…
Browse files Browse the repository at this point in the history
…filter-authenticators-not-supported-by-os
  • Loading branch information
balazs-gerlei authored May 29, 2024
2 parents fdd4cf4 + a2365c1 commit cf7c865
Showing 1 changed file with 6 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -96,9 +96,11 @@ class RegistrationAuthenticatorSelectorImpl(
return null
}

// Do not display policy non-compliant authenticators (this includes already registered
// authenticators), nor those not supported by hardware.
return if (authenticator.isSupportedByHardware && context.isPolicyCompliant(authenticator.aaid())) {
// Do not display:
// - policy non-compliant authenticators (this includes already registered authenticators)
// - not hardware supported authenticators.
// - not OS supported authenticators.
return if (authenticator.isSupportedByHardware && authenticator.isSupportedByOs && context.isPolicyCompliant(authenticator.aaid())) {
AuthenticatorItem(
authenticator.aaid(),
true,
Expand All @@ -113,4 +115,4 @@ class RegistrationAuthenticatorSelectorImpl(
}
}
//endregion
}
}

0 comments on commit cf7c865

Please sign in to comment.