feat: Adds full NFC PIN authenticator support to the SIW(Gen2 and Gen3). #4081
Open
praveendurai-okta wants to merge 34 commits into
Open
feat: Adds full NFC PIN authenticator support to the SIW(Gen2 and Gen3). #4081praveendurai-okta wants to merge 34 commits into
praveendurai-okta wants to merge 34 commits into
Conversation
5 tasks
praveendurai-okta
force-pushed
the
pd-KTA-1141263-nfc-support
branch
from
July 11, 2026 02:38
34b0e4c to
8658bc7
Compare
praveendurai-okta
force-pushed
the
pd-KTA-1141263-nfc-support
branch
from
July 16, 2026 15:51
f7bea6b to
96c53b2
Compare
praveendurai-okta
marked this pull request as ready for review
July 17, 2026 19:27
praveendurai-okta
requested review from
Luxu2-Okta,
alexchen-okta,
glenfannin-okta,
jaredperreault-okta,
leemchale-okta and
shuowu-okta
as code owners
July 17, 2026 19:27
praveendurai-okta
force-pushed
the
pd-KTA-1141263-nfc-support
branch
from
July 21, 2026 17:46
979299a to
6f1e7e3
Compare
LazarVuckovic-okta
requested changes
Jul 21, 2026
praveendurai-okta
force-pushed
the
pd-KTA-1141263-nfc-support
branch
from
July 22, 2026 14:51
16a0f11 to
c637e30
Compare
Luxu2-Okta
approved these changes
Jul 22, 2026
Luxu2-Okta
left a comment
Contributor
There was a problem hiding this comment.
LGTM, please make sure to run all the tests as well
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
NFC PIN is a new proximity-based authenticator that allows users to authenticate by tapping an NFC device and entering a PIN.
This PR implements:
poll-based state transitions
Key Design Decisions
BaseOktaVerifyChallengeView(v2) andtransformOktaVerifyFPLoopbackPoll(v3) for LOOPBACK device challengesPasswordRequirementsandPasswordMatchescomponents with NFC-specific labelscurrentAuthenticatorandcurrentAuthenticatorEnrollmentpaths for the recover (Forgot PIN) actionchallenge-pollstep usescontextualData.challenge.valuepath;device-challenge-pollhandled by genericDeviceChallengePollView
PR Checklist
Issue:
Reviewers:
Screenshot/Video:
Gen 2/ V2
Enrollment Flow
https://drive.google.com/file/d/1NPk6VGjxCAJnNS-clf9LtL4JCrV4bzWy/view?usp=drive_link
Verify with Pin Reset
https://drive.google.com/file/d/1ToW86UVMlz4UIP1KvRQjaBCc4kIgznuA/view?usp=drive_link
Launch NFC Flow
https://drive.google.com/file/d/1x16rlDFRn_LEW8ofUmpCfk9TWm2-9VI5/view?usp=drive_link
Gen 3/ V3
Enrollment Flow
https://drive.google.com/file/d/1WfsD8ZA3HSfmROXnSD3GflnlYM4N3mkB/view?usp=drive_link
Verify with Pin Reset
https://drive.google.com/file/d/13PLy6WSaCJKc63y-HrhQavyedmybH3UD/view?usp=drive_link
Launch NFC Flow
https://drive.google.com/file/d/1Vim5ihA4DWTb6DXp5uEYOtQbUu9vEKMl/view?usp=drive_link
Downstream Monolith Build:
https://bacon-go.aue1e.saasure.net/commits?artifact=monolith&page=1&pageSize=6&sha=182662c9bc341b61fcd42e1cd44ebdd37c8781db&tab=main
SIWv3_Build
https://bacon-go.aue1e.saasure.net/commits?artifact=monolith&branch=SIWv3-builder-pd-nfc-pin-182662c9bc341b61fcd42e1cd44ebdd37c8781db&page=1&pageSize=6&tab=main
IE test run
https://bacon-go.aue1e.saasure.net/tasks/CB_MULTI_BS?taskId=c703d520f3ad46029ff982c5b746ad66
https://bacon-go.aue1e.saasure.net/tasks/CB_MULTI_BS?taskId=662393463ce2471d9b2be2b43aa1d696