-
Notifications
You must be signed in to change notification settings - Fork 758
fix(cloudformation): Simplified error message handling from server to client #8431
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
base: master
Are you sure you want to change the base?
fix(cloudformation): Simplified error message handling from server to client #8431
Conversation
322ac32 to
b4d3aa3
Compare
|
2e2cad5 to
6920226
Compare
6920226 to
b9671c6
Compare
| } | ||
|
|
||
| try { | ||
| await resourcesManager.refreshResourceList(resourceTypeNode.typeName) |
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.
We should probably get rid of the try/catch
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.
Removed
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.
unresolved. I added it back because we are expecting the error to be silently moved on from. We are already displaying the error in logs and notifications at another level so for here we just want to silently continue.
a281775 to
b9671c6
Compare
Problem
the calling code
stacks
Solution
Error handling:
Infinite loop fix:
Tests:
feature/xbranches will not be squash-merged at release time.