CARTAvis / carta-backend

Source code repository for the backend component of CARTA, a new visualization tool designed for the ALMA, the VLA and the SKA pathfinders.
https://cartavis.github.io/
GNU General Public License v3.0
22 stars 11 forks source link

Include full path in error messages about nonexistent files (e.g. when loading a workspace) #1354

Open confluence opened 10 months ago

confluence commented 10 months ago

At the moment if a workspace includes a file which is moved or deleted, when the workspace is loaded the error message about this file only contains the filename, not the full path to the file. This is unhelpful if the user has lots of files which have the same name but are in different subdirectories (this has actually come up recently, and seems like a common use case).

The error message should include the full path to the file. If possible (if it's not considered a security risk1) we should ensure that it's the correct absolute filesystem path in the case where the CARTA root is not the filesystem root. The user should be able to copy this path and use it directly for debugging.

  1. My understanding is that we use the CARTA root mostly to avoid showing the user irrelevant system directories, and that we don't rely on it for security, so exposing the real filesystem path to the CARTA root should not be a problem.