public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: "Fabian Grünbichler" <f.gruenbichler@proxmox.com>
To: pve-devel@lists.proxmox.com, Shannon Sterz <s.sterz@proxmox.com>
Subject: partially-applied: [PATCH many v5 0/8] fix #5076: add support for open id audiences
Date: Thu, 07 May 2026 11:35:05 +0200	[thread overview]
Message-ID: <1778145240.kn3w35ba98.astroid@yuna.none> (raw)
In-Reply-To: <20260423133548.349086-1-s.sterz@proxmox.com>

thanks for pulling this across the finish line!

On April 23, 2026 3:35 pm, Shannon Sterz wrote:
> [..]

> proxmox:
> 
> Shannon Sterz (2):
>   fix #5076: openid: add logic to handle OIDC audiences
>   fix #5076: pbs-api-types: add audiences to open id realm config
> 
>  pbs-api-types/src/openid.rs | 26 ++++++++++++++++++++++++++
>  proxmox-openid/src/lib.rs   | 21 +++++++++++++++++++--
>  2 files changed, 45 insertions(+), 2 deletions(-)

applied and bumped these two

> access-control:
> 
> Shannon Sterz (1):
>   fix #5076: auth: open id: add an optional "audiences" field
> 
>  src/PVE/API2/OpenId.pm | 4 ++++
>  src/PVE/Auth/OpenId.pm | 9 +++++++++
>  2 files changed, 13 insertions(+)

not applied yet: waiting for a pve-rs bump to pick up the proxmox-openid
changes

do we need similar changes in pmg-api before the next pmg-rs bump?

> manager:
> 
> Shannon Sterz (1):
>   fix #5076: ui: dc: add an optional "audiences" field for open id
>     realms
> 
>  www/manager6/dc/AuthEditOpenId.js | 9 +++++++++
>  1 file changed, 9 insertions(+)

not applied yet: needs to wait for pve-access-control, since it's the UI
enablement for the backend change

> yew-comp:
> 
> Shannon Sterz (1):
>   fix #5076: auth edit openid: add advanced  "audiences" field
> 
>  src/auth_edit_openid.rs | 2 ++
>  1 file changed, 2 insertions(+)

applied, but not bumped

once it is bumped, PDM will pick up the UI side of changes when it is
next rebuilt/bumped

> datacenter-manager:
> 
> Shannon Sterz (1):
>   fix #5076: api-types/api: support audiences property for open id
>     realms
> 
>  lib/pdm-api-types/src/openid.rs        | 30 +++++++++++++++++++++++++-
>  server/src/api/access/openid.rs        |  8 +++++++
>  server/src/api/config/access/openid.rs |  8 +++++++
>  3 files changed, 45 insertions(+), 1 deletion(-)

applied with version bump of proxmox-openid, but not bumped PDM itself

> backup:
> 
> Shannon Sterz (1):
>   fix #5076: api: support audiences property for open id realms
> 
>  src/api2/access/openid.rs        | 8 ++++++++
>  src/api2/config/access/openid.rs | 8 ++++++++
>  2 files changed, 16 insertions(+)

applied with version bump of proxmox-openid and pbs-api-types, but not
bumped PBS itself

> widget-toolkit:
> 
> Shannon Sterz (1):
>   fix #5076: ui: dc: add an optional "audiences" field for open id
>     realms
> 
>  src/window/AuthEditOpenId.js | 9 +++++++++
>  1 file changed, 9 insertions(+)

not applied yet, needs to wait for the next PBS bump and then get breaks
on old PBS, dependency from new PBS I guess?

> Summary over all repositories:
>   12 files changed, 139 insertions(+), 3 deletions(-)
> 
> -- 
> Generated by murpp 0.10.0
> 
> 
> 
> 
> 




  parent reply	other threads:[~2026-05-07  9:35 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-23 13:35 [PATCH many v5 0/8] fix #5076: add support for open id audiences Shannon Sterz
2026-04-23 13:35 ` [PATCH proxmox v5 1/8] fix #5076: openid: add logic to handle OIDC audiences Shannon Sterz
2026-04-23 13:35 ` [PATCH proxmox v5 2/8] fix #5076: pbs-api-types: add audiences to open id realm config Shannon Sterz
2026-04-23 13:35 ` [PATCH access-control v5 3/8] fix #5076: auth: open id: add an optional "audiences" field Shannon Sterz
2026-04-23 13:35 ` [PATCH manager v5 4/8] fix #5076: ui: dc: add an optional "audiences" field for open id realms Shannon Sterz
2026-04-23 13:35 ` [PATCH yew-comp v5 5/8] fix #5076: auth edit openid: add advanced "audiences" field Shannon Sterz
2026-04-23 13:35 ` [PATCH datacenter-manager v5 6/8] fix #5076: api-types/api: support audiences property for open id realms Shannon Sterz
2026-04-23 13:35 ` [PATCH proxmox-backup v5 7/8] fix #5076: api: " Shannon Sterz
2026-04-23 13:35 ` [PATCH widget-toolkit v5 8/8] fix #5076: ui: dc: add an optional "audiences" field " Shannon Sterz
2026-05-07  9:35 ` Fabian Grünbichler [this message]
2026-05-07 11:11   ` partially-applied: [PATCH many v5 0/8] fix #5076: add support for open id audiences Shannon Sterz
2026-05-07 11:10 ` [PATCH pmg-api v5] fix #5076: auth: openid: add support for an optional "audiences" field Shannon Sterz
2026-05-07 11:10   ` [PATCH pmg-gui v5] fix #5076: auth edit open id: add an optional audiences field Shannon Sterz

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=1778145240.kn3w35ba98.astroid@yuna.none \
    --to=f.gruenbichler@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    --cc=s.sterz@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