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

Add config value to log Throwable and Cause to ErrorResponseConfig #3177

Open
987Nabil opened this issue Sep 26, 2024 · 1 comment
Open

Add config value to log Throwable and Cause to ErrorResponseConfig #3177

987Nabil opened this issue Sep 26, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@987Nabil
Copy link
Contributor

If turned on, logs internal errors including Endpoint API codec errors, which can currently only be configured to be returned as bodies

@987Nabil 987Nabil added the enhancement New feature or request label Sep 26, 2024
@jdegoes
Copy link
Member

jdegoes commented Nov 9, 2024

I think we should do codec errors differently (if the user doesn't specify a way to map them to response) because a user expects them to be handled inside ZIO HTTP and they cannot leak internal details if done properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants