public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: "Shannon Sterz" <s.sterz@proxmox.com>
To: "Shannon Sterz" <s.sterz@proxmox.com>
Cc: pdm-devel@lists.proxmox.com
Subject: Superseded: Re: [PATCH datacenter-manager/yew-comp 0/4] fix adding users manually for openid/ldap/ad realms
Date: Mon, 09 Feb 2026 10:44:48 +0100	[thread overview]
Message-ID: <DGAC1S1W7KVN.1XNSPIP9FJSOY@proxmox.com> (raw)
In-Reply-To: <20260112122420.228501-1-s.sterz@proxmox.com>

Superseded-by: https://lore.proxmox.com/pdm-devel/20260209094349.34075-1-s.sterz@proxmox.com/T/#u

On Mon Jan 12, 2026 at 1:24 PM CET, Shannon Sterz wrote:
> currently, when adding a new user manually through the ui, a password
> needs to be entered for all realms but the pam realm. however, only for
> the pdm/product realm itself do we store a password. so don't require
> these fields for any other realm.
>
> also adds a dummy authenticator implementation for openid realms. this
> is necessary, because looking up whether an authenticator is present is
> used for validating that a realm exists when adding a user. so add a
> dummy authenticator for openid realms.
>
> proxmox-yew-comp:
>
> Shannon Sterz (1):
>   user panel: add a parameter to set the current product's realm
>
>  src/user_panel.rs | 33 ++++++++++++++++++++++++++-------
>  1 file changed, 26 insertions(+), 7 deletions(-)
>
>
> proxmox-datacenter-manager:
>
> Shannon Sterz (3):
>   fix #7182: server: auth: add dummy openid authenticator
>   ui: run cargo fmt
>   ui: set prodcut realm so that the add user dialogs are rendered
>     properly
>
>  server/src/auth/mod.rs                     | 61 ++++++++++++++++++----
>  ui/src/configuration/mod.rs                |  2 +-
>  ui/src/configuration/subscription_panel.rs |  6 +--
>  ui/src/remotes/firewall/tree.rs            |  6 +--
>  4 files changed, 53 insertions(+), 22 deletions(-)
>
>
> Summary over all repositories:
>   5 files changed, 79 insertions(+), 29 deletions(-)
>
> --
> Generated by git-murpp 0.8.1





      parent reply	other threads:[~2026-02-09  9:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-12 12:24 [pdm-devel] " Shannon Sterz
2026-01-12 12:24 ` [pdm-devel] [PATCH yew-comp 1/1] user panel: add a parameter to set the current product's realm Shannon Sterz
2026-01-16  8:30   ` Dominik Csapak
2026-01-16  9:01   ` [pdm-devel] applied: " Dietmar Maurer
2026-01-12 12:24 ` [pdm-devel] [PATCH datacenter-manager 1/3] fix #7182: server: auth: add dummy openid authenticator Shannon Sterz
2026-01-12 12:24 ` [pdm-devel] [PATCH datacenter-manager 2/3] ui: run cargo fmt Shannon Sterz
2026-01-12 12:24 ` [pdm-devel] [PATCH datacenter-manager 3/3] ui: set prodcut realm so that the add user dialogs are rendered properly Shannon Sterz
2026-01-14 12:35   ` Shannon Sterz
2026-02-09  9:44 ` Shannon Sterz [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DGAC1S1W7KVN.1XNSPIP9FJSOY@proxmox.com \
    --to=s.sterz@proxmox.com \
    --cc=pdm-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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