[pdm-devel] [PATCH datacenter-manager 0/2] drop obsolete PBS 3 incompatibility warning

[pdm-devel] [PATCH datacenter-manager 0/2] drop obsolete PBS 3 incompatibility warning

[Christian Ebner]

Some versions of Proxmox Backup Sever 3.x were incompatible with the
ticket parsing implementation in proxmox-login, used by the PBS client
of PDM. This incompatibility has been fixed in proxmox-login 1.0.2, so
add that version as minimum dependency and drop the now outdated warning
message in the remote add wizard.

Christian Ebner (2):
  d/control: depend on proxmox-login 1.0.2
  Revert "ui: add wizard: note that login currently only works for PBS
    4"

 Cargo.toml                         |  2 +-
 debian/control                     |  2 +-
 ui/src/remotes/wizard_page_info.rs | 15 ---------------
 3 files changed, 2 insertions(+), 17 deletions(-)

-- 
2.47.3



_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel

[pdm-devel] [PATCH datacenter-manager 1/2] d/control: depend on proxmox-login 1.0.2

[Christian Ebner]

proxmox-login version 1.0.2 contains commit b0d98294
("login: use `ticket` if both it and `ticket_info` are provided") of
proxmox repo, fixing ticket parsing issues for some versions of
Proxmox Backup Server 3. This therefore lead to the API token
creation failing with and authentication error when adding the PBS as
remote.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>
---
 Cargo.toml     | 2 +-
 debian/control | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Cargo.toml b/Cargo.toml
index f88025e..f820409 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -46,7 +46,7 @@ proxmox-io = "1.0.1" # tools and client use "tokio" feature
 proxmox-ldap = { version = "1.1", features = ["sync"] }
 proxmox-lang = "1.1"
 proxmox-log = "1"
-proxmox-login = "1"
+proxmox-login = "1.0.2"
 proxmox-rest-server = "1"
 # some use "cli", some use "cli" and "server", pbs-config uses nothing
 proxmox-router = { version = "3.0.0", default-features = false }
diff --git a/debian/control b/debian/control
index e51fc5d..7d32741 100644
--- a/debian/control
+++ b/debian/control
@@ -58,7 +58,7 @@ Build-Depends: cargo:native,
                librust-proxmox-ldap-1.1+default-dev,
                librust-proxmox-ldap-1.1+sync-dev,
                librust-proxmox-log-1+default-dev,
-               librust-proxmox-login-1+default-dev,
+               librust-proxmox-login-1+default-dev (>= 1.0.2-~~),
                librust-proxmox-network-api-1+default-dev,
                librust-proxmox-network-api-1+impl-dev,
                librust-proxmox-product-config-1+default-dev,
-- 
2.47.3



_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel

[pdm-devel] [PATCH datacenter-manager 2/2] Revert "ui: add wizard: note that login currently only works for PBS 4"

[Christian Ebner]

This reverts commit f9b6ba357f52a2cfd122e1318dc547b742c88541.

proxmox-login version 1.0.2 fixes the compatibility issues and PDM
depends on it, so the warning message in the remote add wizard is
obsolete.
---
 ui/src/remotes/wizard_page_info.rs | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/ui/src/remotes/wizard_page_info.rs b/ui/src/remotes/wizard_page_info.rs
index b6ad5b3..7fffdd1 100644
--- a/ui/src/remotes/wizard_page_info.rs
+++ b/ui/src/remotes/wizard_page_info.rs
@@ -379,21 +379,6 @@ impl Component for PdmWizardPageInfo {
         let content = Column::new()
             .class(FlexFit)
             .with_child(Row::new().with_child(input_panel))
-            .with_child(
-                Row::new()
-                    .padding(2)
-                    .gap(2)
-                    .class(css::AlignItems::Center)
-                    .with_optional_child(
-                        if props.remote_type == RemoteType::Pbs && self.user_mode {
-                            Some(tr!(
-                                "Note: Login currently requires Proxmox Backup Server version 4."
-                            ))
-                        } else {
-                            None
-                        },
-                    ),
-            )
             .with_child(
                 Row::new()
                     .padding(2)
-- 
2.47.3



_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel

[pdm-devel] applied: [PATCH datacenter-manager 0/2] drop obsolete PBS 3 incompatibility warning

[Thomas Lamprecht]

On Mon, 06 Oct 2025 13:33:09 +0200, Christian Ebner wrote:
> Some versions of Proxmox Backup Sever 3.x were incompatible with the
> ticket parsing implementation in proxmox-login, used by the PBS client
> of PDM. This incompatibility has been fixed in proxmox-login 1.0.2, so
> add that version as minimum dependency and drop the now outdated warning
> message in the remote add wizard.
> 
> Christian Ebner (2):
>   d/control: depend on proxmox-login 1.0.2
>   Revert "ui: add wizard: note that login currently only works for PBS
>     4"
> 
> [...]

Applied, thanks!

[1/2] d/control: depend on proxmox-login 1.0.2
      commit: 3555c299e8d29c4fd9b0eb8d371e597cb0449826
[2/2] Revert "ui: add wizard: note that login currently only works for PBS 4"
      commit: da396eadc19ab38dd94e95c3ff6c230153ce476e


_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel