- 
                Notifications
    
You must be signed in to change notification settings  - Fork 32
 
🎨 [Frontend] Handle LongRunningTask errors better #6485
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
Conversation
        
          
                services/static-webserver/client/source/class/osparc/data/Resources.js
              
                Outdated
          
            Show resolved
            Hide resolved
        
      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.
👍
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.
In the video attached to the PR, are you using the latest version of the stack, including #6444 ? In theory, after that commit, unhandled exceptions are transformed into 500 with json-response as
{
  "error": {
      "status": 500, 
      "message": "We apologize for the inconvenience. Our team has recorded the issue and is working to resolve it as quickly as possible. Thank you for your patience [OEC: 1234]"
   }
}and on the web-server logs you should get a comprehensive log with details of the exception including the context
          
 Nope, the commit wasn't in yet.  | 
    
          
 | 
    



What do these changes do?
Related issue/s
follows #6444
How to test
Dev-ops checklist