storage: Show better error message

Tests:

- In a VM, mount a loopback disk. Insert an error in ejected code and notice
that HTML message is shown properly during eject operation.

Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
Reviewed-by: James Valleroy <jvalleroy@mailbox.org>
This commit is contained in:
Sunil Mohan Adapa 2024-03-18 14:41:36 -07:00 committed by James Valleroy
parent 598d7570ff
commit 64d754701f
No known key found for this signature in database
GPG Key ID: 77C0C75E7B650808

View File

@ -14,7 +14,7 @@ from django.views.decorators.http import require_POST
from plinth import views
from plinth.modules import storage
from . import get_error_message, privileged
from . import privileged
logger = logging.getLogger(__name__)
@ -90,12 +90,6 @@ def eject(request, device_path):
else:
messages.success(request, _('Device can be safely unplugged.'))
except Exception as exception:
message = get_error_message(exception.args[-2].decode()) # stdout
logger.error('Error ejecting device - %s', message)
messages.error(
request,
_('Error ejecting device: {error_message}').format(
error_message=message))
views.messages_error(request, _('Error ejecting device.'), exception)
return redirect(reverse('storage:index'))