Nordnet: håndtér Signicats nye CPR-trin efter MitID-godkendelsen - #17
Open
Mikrdk wants to merge 1 commit into
Open
Nordnet: håndtér Signicats nye CPR-trin efter MitID-godkendelsen#17Mikrdk wants to merge 1 commit into
Mikrdk wants to merge 1 commit into
Conversation
Efter MitID-godkendelsen viser Signicat nu (siden sommeren 2026) en ekstra side, der beder om CPR-nummer, før login-koden udleveres. Uden dette trin fejler nordnet.py med KeyError paa 'code'. - CPR-nummeret indtastes skjult i terminalen og sendes kun til mitid.dk - Sendes som siden selv goer det (ét 'cpr'-felt til verify-adressen) - Viser antal resterende forsoeg ved forkert nummer (siden kan laase) - Login-koden vaelges nu fra nordnet.dk-adressen i omdirigeringskaeden, da CPR-trinnet tilfoejer et ekstra mitid.dk-stop med sin egen kode
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
1 participant
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.
Tak for det hurtige svar i #16! Her er PR'en.
Hvad den gør
Nordnet-login fejler i øjeblikket efter MitID-godkendelsen, fordi Signicat har indsat en ekstra side, der beder om CPR-nummer, før login-koden udleveres. Denne PR lærer
login_flows/nordnet.pyat håndtere den side:cpr-felt til sidens "verify"-adresse). Svarer siden, at nummeret er forkert, vises hvor mange forsøg der er tilbage — siden låser nemlig efter få forkerte forsøg.Test
Selve fremgangsmåden (alle fire punkter) er verificeret mod rigtige Nordnet-logins med MitID d. 18.-19. juli 2026 i vores eget projekt, hvor en tilsvarende implementering kører i drift. Den konkrete fil her kompilerer og følger repoets eksisterende stil (samme fejlbeskeder som resten af filen) — men jeg har ikke kørt netop denne fil mod et live login, så sig endelig til, hvis noget skal justeres.
Lukker #16.
PR'en er lavet med hjælp fra en AI-assistent (Claude) og gennemgået af mig.