-
Notifications
You must be signed in to change notification settings - Fork 319
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
NAS-128858 / 24.10 / Fix & improve http error handler #10300
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look reasonable, thanks.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #10300 +/- ##
==========================================
+ Coverage 76.70% 76.71% +0.01%
==========================================
Files 1594 1594
Lines 54835 54839 +4
Branches 6477 6479 +2
==========================================
+ Hits 42059 42072 +13
+ Misses 12776 12767 -9 ☔ View full report in Codecov by Sentry. |
This PR has been merged and conversations have been locked. |
Summary
This PR contains 2 changes:
The error modal window depicted in the ticket (
Error(0)
) was shown due to the fact that the handler received not an HTTP error, but a Job error from thewebsocket
. I've fixed this handler.Adjust HTTP error handler to show error title and message from HTTP errors, which was preventing from determining the root cause of the issue
Testing
On any box, for example on http://10.220.16.103
When user clicks Data Protection > Cloud Sync Tasks > Green "Success" Pill
At the error modal window, user clicks Download Logs