Skip to content

DEB-1272 Fixed

Tom Adam requested to merge DEB-1272-loanerId-patronId-crash into master
  • Added an overridden UserManager, that checks Cicero user storage provider as first option if the username only contains numbers. This has altered the original Keycloak approach, which did a local lookup at first causing faulty user data returned when the user tried to log in with a library card number matching another user's patronId. (Locally the patronId is stored as username, thus a lookup with a loner ID matching another patronId will return another user from the local database).
  • Also fixed a bug - user with bad status could log in through ID-porten if the user was registered before state change.
  • Minimized request towards Cicero for one occurrence per user session (previously it was running 3-5 rounds).
  • Fixed CiceroUserStorageProvider::getUserById - previous implementation was not sound.
  • User enabled state is set depending on the status flag in Cicero. A new login will always override a locally set flag!

Merge request reports