Skip to content

redis manager, verbose error logging #1479

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Eugnee
Copy link
Contributor

@Eugnee Eugnee commented Jul 28, 2025

Sometimes it's very useful to see exact error from redis in logs

@miguelgrinberg
Copy link
Owner

Can we instead print the exception message? The stack trace here is not very useful. My concern is that for every person who wants more verbosity in the logs there's going to be another that complains that the logs are too verbose after this change is made.

@Eugnee
Copy link
Contributor Author

Eugnee commented Jul 28, 2025

I usually prefer full stack trace, but you probably right.
In this case, I guess something like str(exc) should be fine.

Sometimes it's very useful to see exact error from redis in logs
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.

2 participants