Bug #5092
En allerede oprettet låner som prøver at oprette sig skal redirectes til indlogningsflowet
Beskrivelse
Problemstilling¶
Lånerne glemmer nogle gange at de allerede er oprettede som lånere på et bibliotek.
Når de igangsætter brugeroprettelsesflowet giver det derfor en uhensigsmæssig brugeroplevelse.
Når man efterfølgende trykker på log ind knappen er man faktisk allerede logget ind, men man får dog fejlmeddelelsen:
<h3>access denied: insufficient permissions</h3><p>access denied: insufficient permissions</p>
Formål¶
Log lånere ind som allerede er kendt når de initierer brugeroprettelsesflowet.
Løsningsforslag¶
Log brugere ind hvis de er oprettede som lånere på biblioteket når de prøver at oprette sig med NemID.
Filer
| opret flow eksisterende bruger.PNG (37,9 KB) opret flow eksisterende bruger.PNG | |||
| opret flow eksisterende bruger 2.PNG (29,9 KB) opret flow eksisterende bruger 2.PNG | |||
| ding_registration fejl.PNG (21 KB) ding_registration fejl.PNG | Watchdog fejlrapport | ||
| gennemført.png (53,6 KB) gennemført.png | |||
| Skærmbillede 2022-05-31 kl. 11.35.28.png (56,3 KB) Skærmbillede 2022-05-31 kl. 11.35.28.png |
Relaterede sager
Opdateret af Rolf Madsen for 
Opdateret af Steen Larsen for cirka 3 år siden
- duplikater Bug #4662: Workflow ved nemid-oprettelse af eksisterende låner tilføjet
Opdateret af Steen Larsen for cirka 3 år siden
- relaterer til Bug #4942: Fejlside 403 viser html-tags tilføjet
Opdateret af Thomas Hansen for mere end 2 år siden
Opdateret af Ninna Rasmussen for mere end 2 år siden
- Status ændret fra Ready for development til Needs code review
Opdateret af Ninna Rasmussen for mere end 2 år siden
- Tildelt til ændret fra Ninna Rasmussen til Gitte Barlach
Opdateret af Gitte Barlach for mere end 2 år siden
- Tildelt til ændret fra Gitte Barlach til Jørgen Nielsen
Opdateret af Jørgen Nielsen for mere end 2 år siden
- Status ændret fra Needs code review til Reviewed
- Tildelt til ændret fra Jørgen Nielsen til Gitte Barlach
reviewet og godkendt
Opdateret af Kasper Garnæs for mere end 2 år siden
- Status ændret fra Reviewed til Integrationstest (Versioner)
- Tildelt til ændret fra Gitte Barlach til Kasper Garnæs
Merged.
Opdateret af Kasper Garnæs for mere end 2 år siden
- Tildelt til ændret fra Kasper Garnæs til Gitte Barlach
Opdateret af Agnete Moos for mere end 2 år siden
- Fil opret flow eksisterende bruger.PNG opret flow eksisterende bruger.PNG tilføjet
- Fil opret flow eksisterende bruger 2.PNG opret flow eksisterende bruger 2.PNG tilføjet
- Tildelt til ændret fra Gitte Barlach til Agnete Moos
Testet på soenderborg-dev.ddbcms.dk med 7.x-6.4.0-beta-1 i Edge
Når jeg (der er oprettet som bruger) forsøger at oprette mig som bruger med NemID på Sønderborgs site som er opgraderet til 7.x-6.4.0-beta-1 er fejlen desværre ikke rettet.
Opdateret af Gitte Barlach for mere end 2 år siden
- Tildelt til ændret fra Agnete Moos til Ninna Rasmussen
Opdateret af Ninna Rasmussen for mere end 2 år siden
- Status ændret fra Integrationstest (Versioner) til Reviewed - Needs info/rework
Opdateret af Ninna Rasmussen for mere end 2 år siden
- Tildelt til ændret fra Ninna Rasmussen til Thomas Hansen
Opdateret af Thomas Hansen for mere end 2 år siden
Altså, er der nogen der lige gider at cheke hvad der ligger i watchdog'en på `ding_registration` efter at fejlen opstår? ding_registration_acceptance_form_submit() logger rent faktisk hvad det er for en fejl den får, og det er tilsyneladende ikke en DingProviderUserExistsError.
Det vil spare mig for et par dage på at forsøge at få login til at virke...
Opdateret af Ninna Rasmussen for mere end 2 år siden
- Tildelt til ændret fra Thomas Hansen til Agnete Moos
Opdateret af Agnete Moos for mere end 2 år siden
- Fil ding_registration fejl.PNG ding_registration fejl.PNG tilføjet
I watchdog ligger følgende fejl på ding_registration. Se også skærmdump.
Reload\Prancer\SwaggerApiError: Unexpected status code from service in /rest/external/v1/DK-754000/patrons. i Reload\Prancer\SwaggerApiRequest->execute() (linje 147 af /var/aegir/platforms/soenderborg210930110003/profiles/ding2/modules/fbs/prancer/SwaggerApiRequest.php).
Opdateret af Agnete Moos for mere end 2 år siden
- Tildelt til ændret fra Agnete Moos til Thomas Hansen
Opdateret af Thomas Hansen for mere end 2 år siden
Se se, så kom vi jo meget længere. Prublemet er at FBS ikke svarer med en dokumenteret kode. Dvs. at FBS ikke svarer med 200-299, 400, 401 eller 404. Man kunne jo frygte at det var en 500.
Spørgsmålet er hvordan vi kommer videre her. Jeg kan ikke debugge lokalt for det kræver at jeg har adgang til en FBS og en bruger at logge ind med og en adgangsplatformen der er med på legen.
Hvis jeg kan få adgang til serveren (med rettigheder til at pille nede i `/var/aegir/platforms/soenderborg210930110003/`) så kan jeg sætte noget debugging ind der kan gøre os klogere på hvad FBS egentligt svarer.
Opdateret af Thomas Hansen for mere end 2 år siden
- Tildelt til ændret fra Thomas Hansen til Ninna Rasmussen
Opdateret af Agnete Moos for mere end 2 år siden
- relaterer til Bug #5272: Fejlbesked ved brugeroprettelse med NemID tilføjet
Opdateret af Ninna Rasmussen for mere end 2 år siden
- Tildelt til ændret fra Ninna Rasmussen til Thomas Hansen
Opdateret af Jesper Kristensen for mere end 2 år siden
- Status ændret fra Reviewed - Needs info/rework til Needs code review
- Tildelt til ændret fra Thomas Hansen til Gitte Barlach
Opdateret af Gitte Barlach for mere end 2 år siden
- Tildelt til ændret fra Gitte Barlach til Jørgen Nielsen
Opdateret af Jørgen Nielsen for mere end 2 år siden
- Status ændret fra Needs code review til Reviewed - Needs info/rework
- Tildelt til ændret fra Jørgen Nielsen til Jesper Kristensen
Noten skal måske lige opdateres?
Opdateret af Thomas Hansen for mere end 2 år siden
Er det for meget forlang at FBS holder deres swagger spec up to date så vi kunne regenerere vores klient og så evt få en payload?
Og nogen der kan forklare forskellen mellem status 409 og CPR_NUMBER_EXISTS?
Opdateret af Jørgen Nielsen for mere end 2 år siden
- Status ændret fra Reviewed - Needs info/rework til Reviewed
- Tildelt til ændret fra Jesper Kristensen til Gitte Barlach
reviewet og godkendt
Opdateret af Kasper Garnæs for mere end 2 år siden
- Status ændret fra Reviewed til Integrationstest (Versioner)
- Tildelt til ændret fra Gitte Barlach til Kasper Garnæs
Merged.
Opdateret af Agnete Moos for mere end 2 år siden
- Fil gennemført.png gennemført.png tilføjet
Testet på soenderborg-dev.ddbcms.dk med 7.x-6.4.0-beta3
1. Jeg trykker på opret bruger.
2. Jeg logger ind med Nemid
3. Jeg indtaster personlige oplysninger
4. Jeg får nedenstående besked og bliver logget ind (se skærmdump)
Gennemført
Du er nu oprettet som bruger.
Det er jo ikke helt korrekt, da jeg jo allerede er oprettet som bruger.
Opdateret af Kasper Garnæs for mere end 2 år siden
- Status ændret fra Integrationstest (Versioner) til Ready for development
- Tildelt til ændret fra Kasper Garnæs til Gitte Barlach
Gitte har bedt om at få ændringer lavet ifm. dette issue rullet tilbage.
Det er sket i dette PR: https://github.com/ding2/ding2/pull/1838
Opdateret af Gitte Barlach for mere end 2 år siden
- Tildelt til ændret fra Gitte Barlach til Ninna Rasmussen
- Udgave ændret fra Release 34 - Bugfixes til Release 35
Vi skal have en udvikler til at se på den igen.
Ideen med at lade en allerede oprettet bruger glide ind i brugeroprettelsesflowet og dermed inddatere sine oplysninger (igen) incl. pinkode kunne egentligt godt have fungeret, såfremt FBS så ville opdatere de allerede eksisterende oplysninger om brugeren. Men det sker desværre ikke. Derfor vil en bruger der allerede er oprettet nemt kunne havne i en situation, hvor han/hun angiver en pinkode i oprettelsesflowet, der afviger fra den, FBS allerede har. Da FBS ikke opdateres, vil den "nye" pinkode ikke virke efterfølgende. (ligesom evt. ny email og afhentningsfillial heller ikke vil slå igennem)
Forslag til løsning er: at en allerede oprettet bruger, der prøver at oprette sig igen, uden videre logges ind, samt at brugeren dernæst ledes hen til brugerprofilen med opfordring til at kontrollere oplysningerne der, herunder opdatere pinkoden (da en bruger, som har glemt han/hun allerede er oprettet sandsynligvis også har glemt sin pinkode)
Hjælpeteksten kunne formuleres noget a la: Du er allerede oprettet som bruger, og er derfor logget ind. Kontroller om dine oplysninger på din brugerprofil er rigtige, og opdater din pinkode.
Opdateret af Ninna Rasmussen for mere end 2 år siden
- Tildelt til ændret fra Ninna Rasmussen til Thomas Hansen
Opdateret af Thomas Hansen for mere end 2 år siden
- Status ændret fra Ready for development til Needs code review
- Tildelt til ændret fra Thomas Hansen til Ninna Rasmussen
https://github.com/ding2/ding2/pull/1849
Genopliver alt det som Kasper revertede og tilføjer en besked og redirect.
Opdateret af Ninna Rasmussen for mere end 2 år siden
- Tildelt til ændret fra Ninna Rasmussen til Gitte Barlach
@Gitte: Vil du efterprøe på Platform.SH?
Opdateret af Gitte Barlach for mere end 2 år siden
Testet på https://5092-user-login-tuhiomi-ton6gvqucinhw.eu-4.platformsh.site/, som er sat op til at simulere Aarhus.
jeg har prøvet lidt at hvert for at få lov til at logge ind med min biblioteksbruger - uden held:
logget ud af alle platform.sh-sites og bibliotekssites, anvendt incognito fuktionen i browseren, startet en anden browser op, end den jeg normalt anvender til test, anvendt en pc der ikke tidligere har været logget ind på noget bibliotekssite. Ligemeget hvad jeg prøver får jeg denne meddelelse, når jeg prøver at logge ind:
Du er ved at blive logget ud af dine bibliotekstjenester¶
Det har ikke været muligt at logge dig ud af alle dine bibliotekstjenester. Du skal istedet lukke din browser for at logge helt ud.
Jeg kan ikke rigtigt gennemskue, hvad der går galt.
Opdateret af Gitte Barlach for mere end 2 år siden
- Status ændret fra Needs code review til Reviewed - Needs info/rework
- Tildelt til ændret fra Gitte Barlach til Ninna Rasmussen
Opdateret af Ninna Rasmussen for cirka 2 år siden
- Udgave ændret fra Release 35 til Release 36
Flyttes til rel. 36, da release 35 holdes alene til NETS EASY-integration. Releasen forventes tidsmæssigt dog ikke meget senere end rel. 35.
Opdateret af Ninna Rasmussen for cirka 2 år siden
- Tildelt til ændret fra Ninna Rasmussen til Thomas Hansen
Opdateret af Ninna Rasmussen for cirka 2 år siden
- Tildelt til ændret fra Thomas Hansen til Ninna Rasmussen
Opdateret af Ninna Rasmussen for cirka 2 år siden
- Tildelt til ændret fra Ninna Rasmussen til Árni Loftsson
Opdateret af Gitte Barlach for cirka 2 år siden
- Status ændret fra Reviewed - Needs info/rework til Needs code review
- Tildelt til ændret fra Árni Loftsson til Jørgen Nielsen
Opdateret af Jørgen Nielsen for næsten 2 år siden
- Status ændret fra Needs code review til Reviewed
- Tildelt til ændret fra Jørgen Nielsen til Gitte Barlach
reviewet og godkendt
Opdateret af Kasper Garnæs for næsten 2 år siden
- Tildelt til ændret fra Gitte Barlach til Kasper Garnæs
Jeg har merged https://github.com/ding2/ding2/pull/1849
Opdateret af Kasper Garnæs for næsten 2 år siden
- Status ændret fra Reviewed til Integrationstest (Versioner)
Opdateret af Kasper Garnæs for næsten 2 år siden
- Tildelt til ændret fra Kasper Garnæs til Gitte Barlach
Opdateret af Gitte Barlach for næsten 2 år siden
Testet på upgrade-fbs med
a) en allerede oprettet bruger, der prøver at oprette sig via NEMID (linket "Ny bruger?" på https://login.bib.dk/login ) bliver efter at have gennemført oprettelsesflow logget ind. Bruger kommer ind på brugerprofilen-siden, og ser flg. besked: "You already have an account, so you have been logged in. Please check your profile information, and update your pincode."
b) vi mangler en oversættelse af flg.: You already have an account, so you have been logged in. Please check your profile information, and update your pincode.
Opdateret af Gitte Barlach for næsten 2 år siden
- Kategorier Driftsvedligehold - Oversættelser (udarbejdes før udrulning) tilføjet
Opdateret af Gitte Barlach for næsten 2 år siden
- Status ændret fra Integrationstest (Versioner) til Resolved (tag version)
Hermed godkendt.