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

All errors should be logged #1407

Open
mausch opened this issue Apr 2, 2020 · 1 comment
Open

All errors should be logged #1407

mausch opened this issue Apr 2, 2020 · 1 comment
Milestone

Comments

@mausch
Copy link

@mausch mausch commented Apr 2, 2020

The title might seem a bit vague but I don't know how to describe it any better tbh :-)

Anyway this is what happened: I got some 500 responses from the schema registry and all I could see in the logs was :

[2020-04-02 16:03:35,048] INFO 100.96.14.58 - - [02/Apr/2020:16:03:34 +0000] "PUT /config/some-topic-value HTTP/1.1" 500 69  502 (io.confluent.rest-utils.requests)

The logs did not have any stack trace or anything pointing to what caused the error, so there's nothing I can do to start investigating its cause.
All errors should be logged with a stack trace, do you agree?
I'm on version 5.3.3, deployed with the helm chart.

@tusharnt tusharnt added this to the H1 2020 milestone Apr 3, 2020
@OneCricketeer
Copy link
Contributor

@OneCricketeer OneCricketeer commented Apr 6, 2020

500 is an internal server error. It's bad practice to return internal errors to clients.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.