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

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

[Alexander Abraham]

A field was added to the configuration prompt for
configuring audiences in OpenID realms.

Signed-off-by: Alexander Abraham <a.abraham@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() {
-- 
2.39.5



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

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

[Dominik Csapak]

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@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() {



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

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

[Alexander Abraham]

Superseded by: https://lore.proxmox.com/pve-devel/20250602141458.142000-1-a.abraham@proxmox.com/T/#t
> Dominik Csapak <d.csapak@proxmox.com> hat am 17.02.2025 10:00 CET geschrieben:
> 
>  
> 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@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() {


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