NLRFIM-121: Session handling and storage in Java OIO SAML 3

digst / OIOSAML.Java

Other

11 stars 11 forks source link

NLRFIM-121: Session handling and storage in Java OIO SAML 3 #66

Closed TauBeblein closed 2 years ago

TauBeblein commented 2 years ago

NLRFIM-120: Support SOAP bindings in Java OIO SAML 3
NLRFIM-121: Session handling and storage in Java OIO SAML 3

TauBeblein commented 2 years ago

Added unit test in-memory database to unit test database session handler - due to the structure of the project it's necessary.

Look at test coverage, is this pull request enough?

If it is, then documentation is the only part missing!

mdbech commented 2 years ago

Dokumentation review og godkendt.

mdbech commented 2 years ago

Selvom vi benytter Servlet version 3 så tror jeg det er fint at lade det være op til TU'erne selv at beslutter hvordan de rydder op på sessioner og dermed hvordan de vil gerne vil registrerer dem. Så jeg synes ikke vi skal konverterer til at benyttes WebListener for SessionDestroyedListener, men vi kan ved senere lejlighed overveje det for måden hvorpå konfiguration læses ind på.

TauBeblein commented 2 years ago

Alle review ændringer er implementeret, det er klar til review.