Skip to content

Bugfix 404 response from getFlowError#193

Closed
eh-steve wants to merge 2 commits intoory:mainfrom
eh-steve:bugfix-error-redirect
Closed

Bugfix 404 response from getFlowError#193
eh-steve wants to merge 2 commits intoory:mainfrom
eh-steve:bugfix-error-redirect

Conversation

@eh-steve
Copy link

@eh-steve eh-steve commented Aug 5, 2024

Since kratos has only implemented the /self-service/errors?id=some_error_id endpoint for id=stub:500, the current behaviour of the errors page is to always redirect to a 404 error instead of showing the original error.

This PR makes it preserve the original error if kratos returns a 404

Checklist

  • I have read the contributing guidelines and signed the CLA.
  • I have referenced an issue containing the design document if my change introduces a new feature.
  • I have read the security policy.
  • I confirm that this pull request does not address a security vulnerability.
    If this pull request addresses a security vulnerability,
    I confirm that I got approval (please contact security@ory.sh) from the maintainers to push the changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added the necessary documentation within the code base (if appropriate).

@CLAassistant
Copy link

CLAassistant commented Aug 5, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ eh-steve
❌ Anonymous


Anonymous seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@eh-steve eh-steve force-pushed the bugfix-error-redirect branch from 9049581 to a46048e Compare August 5, 2024 14:59
@aeneasr
Copy link
Member

aeneasr commented May 19, 2025

Thank you for your contribution! We are moving to a new version of Ory Elements, where this is no longer a problem. Thus, I'm closing PRs against the old elements, which you can now find here: https://github.com/ory/elements-legacy

@aeneasr aeneasr closed this May 19, 2025
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