all lists on 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: [pdm-devel] Superseded: Re: [PATCH datacenter-manager/yew-comp v2 0/5] add better token support for pdm
Date: Fri, 17 Oct 2025 14:48:32 +0200	[thread overview]
Message-ID: <DDKLXQ7Y6MK7.BM8DX6V0WWMG@proxmox.com> (raw)
In-Reply-To: <20251014143709.413690-1-s.sterz@proxmox.com>

On Tue Oct 14, 2025 at 4:37 PM CEST, Shannon Sterz wrote:
> this series aims to add a ui to the pre-existing token support in pdm.
> it also aims to get it more in-line with what other proxmox products
> provide in terms of functionality.
>
> the first two commits improve the token panel in proxmox-yew-comp by
> using the Clipboard API when copying values. the layout and
> functionality of the dialog displaying the token secret is also
> improved.
>
> the final three patches integrate the token panel in pdm's ui. they also
> refactor the api endpoints related token handling to use the new
> endpoints from proxmxo-access-control and make sure the user delete
> endpoint cleans up acls and tokens too.
>
> Changelog
> ---------
>
> changes since v1:
> - the patches for proxmox-access-control got applied (thanks @ Thomas
>   Lamprecht)
> - use `.gap(2)` instead of `.margin_start(2)` (thanks @ Dominik Csapak)
> - use `ColorScheme::WarningContainer` and `pwt-default-color` instead of
>   manually setting background and color classes (thanks @ Dominik
>   Csapak)
> - rebased on current master branches
>
> changes since rfc:
> - the commits implementing the basic token panel have already been
>   applied by Thomas Lamprecht, thanks!
> - moved adding `use` and `mod` statements for the token module to the
>   right commit in the series (thanks @ Dominik Csapak)
> - generate token secrets in the `token_shadow` module instead of in the
>   token endpoints themselves (thanks @ Fabian Grünbichler)
> - use a schema for the `regenerate` parameter of the update token
>   endpoint (thanks @ Fabian Grünbichler)
> - allow deleting comments via a `delete` property (thanks @ Fabian
>   Grünbichler)
> - make the token delete endpoint clean up token acls (thanks @ Fabian
>   Grünbichler)
> - improve copy to clipboard functionality to use the new Clipboard API
> - improve the layout of the token secret dialog (thanks @ Thomas
>   Lamprecht)
>
> proxmox-yew-comp:
>
> Shannon Sterz (2):
>   utils/tfa add recover/token panel: add copy_text_to_clipboard function
>   token panel: improve token secret dialog layout and hide password
>
>  Cargo.toml                  |   2 +
>  src/tfa/tfa_add_recovery.rs |  17 ++----
>  src/token_panel.rs          | 117 ++++++++++++++++++------------------
>  src/utils.rs                |  22 +++++++
>  4 files changed, 89 insertions(+), 69 deletions(-)
>
>
> proxmox-datacenter-manager:
>
> Shannon Sterz (3):
>   ui: add a token panel and a token acl edit menu in the permissions
>     panel
>   server: access: use token endpoints from proxmox-access-control
>   server: clean up acl tree entries and api tokens when deleting users
>
>  server/src/api/access/users.rs | 388 ++++++---------------------------
>  ui/src/configuration/mod.rs    |  33 ++-
>  2 files changed, 95 insertions(+), 326 deletions(-)
>
>
> Summary over all repositories:
>   6 files changed, 184 insertions(+), 395 deletions(-)
>
> --
> Generated by git-murpp 0.8.1

Superseded-by: https://lore.proxmox.com/pdm-devel/20251017124646.294343-2-s.sterz@proxmox.com/T/#t


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

      parent reply	other threads:[~2025-10-17 12:48 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-14 14:37 [pdm-devel] " Shannon Sterz
2025-10-14 14:37 ` [pdm-devel] [PATCH yew-comp v2 1/2] utils/tfa add recover/token panel: add copy_text_to_clipboard function Shannon Sterz
2025-10-17  9:35   ` Lukas Wagner
2025-10-17 10:43     ` Shannon Sterz
2025-10-14 14:37 ` [pdm-devel] [PATCH yew-comp v2 2/2] token panel: improve token secret dialog layout and hide password Shannon Sterz
2025-10-17  9:36   ` Lukas Wagner
2025-10-17 10:04     ` Shannon Sterz
2025-10-17 11:03       ` Lukas Wagner
2025-10-14 14:37 ` [pdm-devel] [PATCH datacenter-manager v2 1/3] ui: add a token panel and a token acl edit menu in the permissions panel Shannon Sterz
2025-10-17  9:36   ` Lukas Wagner
2025-10-14 14:37 ` [pdm-devel] [PATCH datacenter-manager v2 2/3] server: access: use token endpoints from proxmox-access-control Shannon Sterz
2025-10-17  9:36   ` Lukas Wagner
2025-10-14 14:37 ` [pdm-devel] [PATCH datacenter-manager v2 3/3] server: clean up acl tree entries and api tokens when deleting users Shannon Sterz
2025-10-17  9:36   ` Lukas Wagner
2025-10-17 12:48 ` 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=DDKLXQ7Y6MK7.BM8DX6V0WWMG@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 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