Skip to content

Added checks on login response required from demo validator#143

Open
SeemoneB wants to merge 3 commits intoitalia:masterfrom
SeemoneB:adding_checks_on_login_response
Open

Added checks on login response required from demo validator#143
SeemoneB wants to merge 3 commits intoitalia:masterfrom
SeemoneB:adding_checks_on_login_response

Conversation

@SeemoneB
Copy link

@SeemoneB SeemoneB commented Nov 10, 2022

Managed the follows checks:

  • "08. Response - ID non specificato";
  • "14. Response - IssueInstant precedente Request";
  • "32. Response - Elemento Assertion mancante";
  • "39. Assertion - Attributo IssueInstant precedente a IssueInstant della Request";
  • "40. Assertion - Attributo IssueInstant successivo a IssueInstant della Request";
  • "42. Assertion - Elemento Subject mancante";
  • "52. Assertion - Elemento SubjectConfirmation mancante";
  • "88. Assertion - Elemento AuthStatement non specificato";
  • "89. Assertion - Elemento AuthStatement mancante";
  • "90. Assertion - Elemento AuthnContext di AuthStatement non specificato";
  • "91. Assertion - Elemento AuthnContext di AuthStatement mancante";
  • "92. Assertion - Elemento AuthContextClassRef di AuthnContext di AuthStatement non specificato";
  • "93. Assertion - Elemento AuthContextClassRef di AuthnContext di AuthStatement mancante";
  • "97. Assertion - Elemento AuthContextClassRef impostato ad un valore non previsto";
  • "103. Assertion - Set di attributi inviato diverso da quello richiesto";

Copy link

@scint001 scint001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! These changes are mandatory for SPID validator

@SeemoneB SeemoneB requested a review from lussoluca March 3, 2023 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants