ui: ldap: add 'Check connection' checkbox as advanced option

The checkbox is enabled by default, setting the new `check-connection`
parameter. See also [0] for the rationale.

[0] https://lists.proxmox.com/pipermail/pve-devel/2023-July/058559.html

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>

diff --git a/www/manager6/dc/AuthEditAD.js b/www/manager6/dc/AuthEditAD.js
index a1999cb7480337a3e413d4e63619a9be793778a2..3cbb47c9f57ddc435ccbdcef450a113f99722d9c 100644 (file)
--- a/www/manager6/dc/AuthEditAD.js
+++ b/www/manager6/dc/AuthEditAD.js
@@ -79,6 +79,21 @@ Ext.define('PVE.panel.ADInputPanel', {
            },
        ];
 
+       me.advancedItems = [
+           {
+               xtype: 'proxmoxcheckbox',
+               fieldLabel: gettext('Check connection'),
+               name: 'check-connection',
+               uncheckedValue: 0,
+               checked: true,
+               autoEl: {
+                   tag: 'div',
+                   'data-qtip':
+                       gettext('Verify connection parameters and bind credentials on save'),
+               },
+           },
+       ];
+
        me.callParent();
     },
     onGetValues: function(values) {

diff --git a/www/manager6/dc/AuthEditLDAP.js b/www/manager6/dc/AuthEditLDAP.js
index 2ce16e58c008314f99afa7f4dd7325ca482a8500..9986db8a9b41c8feaa94911f6a0416244a2705a1 100644 (file)
--- a/www/manager6/dc/AuthEditLDAP.js
+++ b/www/manager6/dc/AuthEditLDAP.js
@@ -79,6 +79,21 @@ Ext.define('PVE.panel.LDAPInputPanel', {
            },
        ];
 
+       me.advancedItems = [
+           {
+               xtype: 'proxmoxcheckbox',
+               fieldLabel: gettext('Check connection'),
+               name: 'check-connection',
+               uncheckedValue: 0,
+               checked: true,
+               autoEl: {
+                   tag: 'div',
+                   'data-qtip':
+                       gettext('Verify connection parameters and bind credentials on save'),
+               },
+           },
+       ];
+
        me.callParent();
     },
     onGetValues: function(values) {