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.

Sign up for GitHub

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

Jump to bottom

Web console: better error reporting #13636

Merged
merged 1 commit into from
Jan 6, 2023
Merged

Web console: better error reporting #13636

merged 1 commit into from
Jan 6, 2023

Conversation

vogievetsky
Copy link
Contributor

Fixes #13635

Make better reporting by trying to extract whatever message exists in the response.

Here is how it looks now:

image

clintropolis
clintropolis approved these changes Jan 6, 2023
View reviewed changes
Copy link
Member

@clintropolis clintropolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice 🚀

@vogievetsky vogievetsky merged commit 4ee4d99 into apache:master Jan 6, 2023
@vogievetsky vogievetsky deleted the better_error_feedback branch January 6, 2023 04:00
@meticulo3366
Copy link

in our case it could connect to the topic but there were no messages in the topic and hence 504 exception

@clintropolis clintropolis added this to the 26.0 milestone Apr 10, 2023
@techdocsmith techdocsmith mentioned this pull request Apr 12, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clintropolis clintropolis clintropolis approved these changes

Assignees
No one assigned
Labels
Area - Web Console Error handling
Projects
None yet
Milestone
26.0
3 participants
@vogievetsky @meticulo3366 @clintropolis