public inbox for pbs-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH proxmox-backup v2] ui: encryption keys: show error text instead of empty popup in API alerts
@ 2026-04-29 10:02 Arthur Bied-Charreton
  2026-04-29 10:13 ` Arthur Bied-Charreton
  0 siblings, 1 reply; 2+ messages in thread
From: Arthur Bied-Charreton @ 2026-04-29 10:02 UTC (permalink / raw)
  To: pbs-devel

The errors returned by the encryption-keys and tape-encryption-keys
endpoints are not plain strings, so the alerts displayed on API2 errors
like failed permission checks end up being empty and therefore not very
helpful.

Extract the error messages to display them correctly in alerts.

Signed-off-by: Arthur Bied-Charreton <a.bied-charreton@proxmox.com>
---
 www/config/EncryptionKeysView.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/www/config/EncryptionKeysView.js b/www/config/EncryptionKeysView.js
index 0f9367c7..bb327814 100644
--- a/www/config/EncryptionKeysView.js
+++ b/www/config/EncryptionKeysView.js
@@ -168,7 +168,7 @@ Ext.define('PBS.config.EncryptionKeysView', {
                     });
                 }
             } catch (error) {
-                Ext.Msg.alert(gettext('Error'), error);
+                Ext.Msg.alert(gettext('Error'), error.result?.message || gettext('Unknown error'));
             }
 
             try {
@@ -181,7 +181,7 @@ Ext.define('PBS.config.EncryptionKeysView', {
                     });
                 }
             } catch (error) {
-                Ext.Msg.alert(gettext('Error'), error);
+                Ext.Msg.alert(gettext('Error'), error.result?.message || gettext('Unknown error'));
             }
 
             let store = view.getStore().rstore;
-- 
2.47.3




^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH proxmox-backup v2] ui: encryption keys: show error text instead of empty popup in API alerts
  2026-04-29 10:02 [PATCH proxmox-backup v2] ui: encryption keys: show error text instead of empty popup in API alerts Arthur Bied-Charreton
@ 2026-04-29 10:13 ` Arthur Bied-Charreton
  0 siblings, 0 replies; 2+ messages in thread
From: Arthur Bied-Charreton @ 2026-04-29 10:13 UTC (permalink / raw)
  To: pbs-devel

superseded by v3:
https://lore.proxmox.com/pbs-devel/20260429101139.163532-1-a.bied-charreton@proxmox.com/T/#u

sorry forgot to fold the htmlStatus change into v2..




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-04-29 10:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-29 10:02 [PATCH proxmox-backup v2] ui: encryption keys: show error text instead of empty popup in API alerts Arthur Bied-Charreton
2026-04-29 10:13 ` Arthur Bied-Charreton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal