all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH pve-manager v2] api: sdn: fix double slashes in delete requests
@ 2026-04-28 11:39 David Riley
  2026-04-28 12:27 ` superseded: " David Riley
  0 siblings, 1 reply; 2+ messages in thread
From: David Riley @ 2026-04-28 11:39 UTC (permalink / raw)
  To: pve-devel; +Cc: David Riley

Fix deletion requests for SDN: DNS, IPAM, and Controller options which
contained a double slash (//). Even though the server normalized
this and the requests succeeded, cleaning it up avoids potential
issues with strict reverse proxies or firewalls.

Signed-off-by: David Riley <d.riley@proxmox.com>
---

Notes:
    Difference to v1: fix IPAM and Controller

 www/manager6/sdn/ControllerView.js | 2 +-
 www/manager6/sdn/DnsView.js        | 2 +-
 www/manager6/sdn/IpamView.js       | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/www/manager6/sdn/ControllerView.js b/www/manager6/sdn/ControllerView.js
index d85e85e4..cd8b4bc0 100644
--- a/www/manager6/sdn/ControllerView.js
+++ b/www/manager6/sdn/ControllerView.js
@@ -60,7 +60,7 @@ Ext.define('PVE.sdn.ControllerView', {
 
         let remove_btn = Ext.create('Proxmox.button.StdRemoveButton', {
             selModel: sm,
-            baseurl: '/cluster/sdn/controllers/',
+            baseurl: '/cluster/sdn/controllers',
             callback: () => store.load(),
         });
 
diff --git a/www/manager6/sdn/DnsView.js b/www/manager6/sdn/DnsView.js
index ba5ada5a..c5cd32bb 100644
--- a/www/manager6/sdn/DnsView.js
+++ b/www/manager6/sdn/DnsView.js
@@ -59,7 +59,7 @@ Ext.define('PVE.sdn.DnsView', {
 
         let remove_btn = Ext.create('Proxmox.button.StdRemoveButton', {
             selModel: sm,
-            baseurl: '/cluster/sdn/dns/',
+            baseurl: '/cluster/sdn/dns',
             callback: () => store.load(),
         });
 
diff --git a/www/manager6/sdn/IpamView.js b/www/manager6/sdn/IpamView.js
index ca9068cf..1764c184 100644
--- a/www/manager6/sdn/IpamView.js
+++ b/www/manager6/sdn/IpamView.js
@@ -58,7 +58,7 @@ Ext.define('PVE.sdn.IpamView', {
 
         let remove_btn = Ext.create('Proxmox.button.StdRemoveButton', {
             selModel: sm,
-            baseurl: '/cluster/sdn/ipams/',
+            baseurl: '/cluster/sdn/ipams',
             callback: () => store.load(),
         });
 
-- 
2.47.3





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

* superseded: [PATCH pve-manager v2] api: sdn: fix double slashes in delete requests
  2026-04-28 11:39 [PATCH pve-manager v2] api: sdn: fix double slashes in delete requests David Riley
@ 2026-04-28 12:27 ` David Riley
  0 siblings, 0 replies; 2+ messages in thread
From: David Riley @ 2026-04-28 12:27 UTC (permalink / raw)
  To: pve-devel

superseded by
https://lore.proxmox.com/pve-devel/20260428122156.82218-1-d.riley@proxmox.com





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

end of thread, other threads:[~2026-04-28 12:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-04-28 11:39 [PATCH pve-manager v2] api: sdn: fix double slashes in delete requests David Riley
2026-04-28 12:27 ` superseded: " David Riley

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal