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

Incorrect documentation regarding "Error Sanitization"? #853

Open
nocin opened this issue Apr 23, 2024 · 3 comments
Open

Incorrect documentation regarding "Error Sanitization"? #853

nocin opened this issue Apr 23, 2024 · 3 comments

Comments

@nocin
Copy link
Contributor

nocin commented Apr 23, 2024

Hi @renejeglinsky,

I would just like to use this issue to reiterate the problem already described by @gregorwolf in the SCN community.

https://community.sap.com/t5/technology-q-a/incorrect-documentation-regarding-quot-error-sanitization-quot/qaq-p/12710155

The documentation and the behavior of the CAP framework do not seem to be entirely consistent in regard to "Error Sanitization".

As a current workaround, if I want to display an error message in the frontend, I always set the http code to 4xx instead of 5xx, even if this is not actually correct.

Best regards,
Nico

@renejeglinsky
Copy link
Contributor

Hi @nocin ,
I'm already in touch with our Node.js runtime team and will get back to you.
All the best,
René

@nocin
Copy link
Contributor Author

nocin commented Oct 10, 2024

Hi @renejeglinsky,

is there any update on this? Currently working on some error handling, where I want the returned errors not to be sanitized, and it's still unclear how this is done properly.

Thanks,
Nico

@johannes-vogel
Copy link
Contributor

Hi @nocin,

there's a longer internal discussion going on.
Could you give us a quick summary of the error you're trying to return?

Best,
Johannes

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

No branches or pull requests

3 participants