Skip to content

unsolicited response specific error#1965

Open
kayjoosten wants to merge 3 commits intomainfrom
feature/unsolicited-response-specific-error
Open

unsolicited response specific error#1965
kayjoosten wants to merge 3 commits intomainfrom
feature/unsolicited-response-specific-error

Conversation

@kayjoosten
Copy link
Copy Markdown
Contributor

@kayjoosten kayjoosten commented Apr 8, 2026

All languages
unsolicited-en
unsolicited-nl
unsolicited-pt

…tiated SSO)

When an IdP sends an assertion without InResponseTo (e.g. via a portal tile,
bookmark, or saved link), EngineBlock now shows a specific user-friendly error
page instead of the generic unknown-error page.

Fixes #1838
… SSO)

- Add omitInResponseTo flag to MockIdentityProvider (serialized in __sleep)
- ResponseFactory: skip InResponseTo when flag set, explicitly null the template placeholder
- MockIdpContext: new step 'the IdP omits InResponseTo from its response'
- Bindings.feature: scenario asserting redirect to unsolicited-response error page
@kayjoosten kayjoosten changed the title Feature/unsolicited response specific error unsolicited response specific error Apr 8, 2026
@kayjoosten
Copy link
Copy Markdown
Contributor Author

@baszoetekouw would you like to see some other message? My spelling isn't the best so I'm not 100% sure these are the correct translations as well.

@johanib johanib linked an issue Apr 9, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@johanib johanib left a comment

Choose a reason for hiding this comment

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

Looks nice & clean! 👍

{% block title %}{{ parent() }} - {{ pageTitle }} {% endblock %}
{% block pageHeading %}{{ pageTitle }}{% endblock %}

{% block errorMessage %}{{ 'error_unsolicited_response_desc'|trans }}{% endblock %}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should the skeune theme also have this template? 🤔

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.

Specific error for unsollicited response

2 participants