Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added checks on login response required from demo validator #143

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

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";

src/Spid/Saml/In/Response.php Outdated Show resolved Hide resolved
src/Spid/Saml/In/Response.php Outdated Show resolved Hide resolved
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