[#IOPID-1612, #IOPID-1513] SAMLResponse missing errorMessage

[arcogabbo]

List of Changes

• custom errorMessage in case SAMLResponse is missing

Motivation and Context

At the moment, the error for a SAMLResponse missing is misleading and it cames from the passport-saml library that tries to construct an AuthNRequest if it doesn't find a proper SAMLResponse in body. This change will prevent that scenario to happen, returning a proper errorMessage so it can be tracked in our dashboards.

How Has This Been Tested?

unit tests, integration test via io-mock on io-backend

on [email protected] with this io-spid-commons build injected:

curl --insecure -X POST https://localhost:8000/assertionConsumerService

## Found. Redirecting to https://localhost:8000/error.html?errorMessage=Missing%20SAMLResponse%20in%20ACS

Screenshots (if appropriate):

Types of changes

• Chore (nothing changes by a user perspective)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.

[github-actions]

Jira Pull Request Link

This Pull Request refers to Jira issues:
- Link to IOPID-1612
- Link to IOPID-1513

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[gquadrati]

LGTM 🚀