Meaningful error messages if InternalServerError is thrown

[AndrejGajdos]

Why do you set Unexpected error when InternalServerError is thrown?

Right now, the actual error messages are not passed to forestadmin.

For example if a record cannot be deleted I can see in console.log:

In ForestAdmin you get Unexpected error and this error message is not very useful.

[jeffladiray]

Hey @AndrejGajdos :wave:

This was initially done deliberately to prevent leaking informations from the database.

However, we missed the fact that this kind of validation errors may be interesting from a frontend perspective. I'll open a ticket in our issue tracker to handle all these cases, and we will keep you updated on the resolution 👍

[romain-gilliotte]

Hi @AndrejGajdos

As said by @jeffladiray, we can't merge your contribution, because we're afraid to leak internal information to users that may be contained in unhandled errors.

I've been assigned to the ticket. How about this?

• 388

• 386

• 390

[AndrejGajdos]

@romain-gilliotte this is great, thanks!

[romain-gilliotte]

The three PRs have been merged, so I'm closing this issue