[pve-devel] [PATCH pve-manager] fix #5076: Added a field for OpenID audiences to the configuration UI

Dominik Csapak d.csapak at proxmox.com
Mon Feb 17 10:00:20 CET 2025


depending on what restrictions in the backend will be implemented,
we should add the same here, so users get immediate feedback
for wrong values

aside from that LGTM

On 2/6/25 13:27, Alexander Abraham wrote:
> A field was added to the configuration prompt for
> configuring audiences in OpenID realms.
> 
> Signed-off-by: Alexander Abraham <a.abraham at proxmox.com>
> ---
>   www/manager6/dc/AuthEditOpenId.js | 9 +++++++++
>   1 file changed, 9 insertions(+)
> 
> diff --git a/www/manager6/dc/AuthEditOpenId.js b/www/manager6/dc/AuthEditOpenId.js
> index 544c0de5..7942fa58 100644
> --- a/www/manager6/dc/AuthEditOpenId.js
> +++ b/www/manager6/dc/AuthEditOpenId.js
> @@ -111,6 +111,15 @@ Ext.define('PVE.panel.OpenIDInputPanel', {
>   		deleteEmpty: '{!isCreate}',
>   	    },
>   	},
> +	{
> +	    xtype: 'proxmoxtextfield',
> +	    name: 'aud',
> +	    fieldLabel: gettext('Audiences'),
> +	    submitEmpty: false,
> +	    cbind: {
> +		deleteEmpty: '{!isCreate}',
> +	    },
> +	},
>       ],
>   
>       initComponent: function() {





More information about the pve-devel mailing list