From a041a155022418acffff1b611a8775b0b1178f24 Mon Sep 17 00:00:00 2001
From: alphaeusmote <41258468-alphaeusmote@users.noreply.replit.com>
Date: Thu, 10 Apr 2025 21:45:40 +0000
Subject: [PATCH] Add option to automatically include rootDSE in OU paths

Replit-Commit-Author: Agent
Replit-Commit-Session-Id: 705f2157-ef97-4fbd-89e4-8c7f2ecaea90
Replit-Commit-Screenshot-Url: https://storage.googleapis.com/screenshot-production-us-central1/7ed01c5f-a82d-405a-b728-b2e3d127c60c/5ed4a3a4-e383-48c9-beb9-321c509bc354.jpg
---
 .../components/dynamic-groups/rule-editor.tsx | 24 ++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

diff --git a/client/src/components/dynamic-groups/rule-editor.tsx b/client/src/components/dynamic-groups/rule-editor.tsx
index 9b40567..0b33d8f 100644
--- a/client/src/components/dynamic-groups/rule-editor.tsx
+++ b/client/src/components/dynamic-groups/rule-editor.tsx
@@ -82,6 +82,7 @@ export const RuleEditor: React.FC<RuleEditorProps> = ({ rule, onSave, onCancel }
     createOUIfNotExists: boolean;
     createGroupForEachAttributeValue: boolean;
     createOUForEachAttributeValue: boolean;
+    autoAddRootDSE: boolean;
     enabled: boolean;
     variablePattern: string;
     connections: number[];
@@ -94,6 +95,7 @@ export const RuleEditor: React.FC<RuleEditorProps> = ({ rule, onSave, onCancel }
     createOUIfNotExists: rule?.createOUIfNotExists || false,
     createGroupForEachAttributeValue: rule?.createGroupForEachAttributeValue || false,
     createOUForEachAttributeValue: rule?.createOUForEachAttributeValue || false,
+    autoAddRootDSE: true, // Default to true for better user experience
     enabled: rule?.enabled ?? true,
     variablePattern: rule?.variablePattern || '',
     connections: rule?.connectionIds || []
@@ -167,10 +169,15 @@ export const RuleEditor: React.FC<RuleEditorProps> = ({ rule, onSave, onCancel }
     return domainParts.map(part => `DC=${part}`).join(',');
   };
   
-  // Ensure OU path has rootDSE appended
+  // Ensure OU path has rootDSE appended if autoAddRootDSE is enabled
   const ensureRootDSE = (ouPath: string) => {
     if (!ouPath) return '';
     
+    // If autoAddRootDSE is disabled, return the path as is
+    if (!formData.autoAddRootDSE) {
+      return ouPath;
+    }
+    
     // Check if the path already contains DC=
     if (ouPath.includes('DC=')) {
       return ouPath;
@@ -474,6 +481,15 @@ export const RuleEditor: React.FC<RuleEditorProps> = ({ rule, onSave, onCancel }
                   />
                   <Label htmlFor="createOUIfNotExists">Create OU if it doesn't exist</Label>
                 </div>
+
+                <div className="flex items-center space-x-2">
+                  <Switch 
+                    id="autoAddRootDSE"
+                    checked={formData.autoAddRootDSE}
+                    onCheckedChange={(checked) => handleToggleChange('autoAddRootDSE', checked)}
+                  />
+                  <Label htmlFor="autoAddRootDSE">Automatically add root DSE to OU path</Label>
+                </div>
                 
                 <div className="space-y-2">
                   <Label htmlFor="targetGroupName">Group Name <span className="text-destructive">*</span></Label>
@@ -636,6 +652,12 @@ export const RuleEditor: React.FC<RuleEditorProps> = ({ rule, onSave, onCancel }
                           {formData.createOUIfNotExists ? 'Yes' : 'No'}
                         </dd>
                       </div>
+                      <div className="flex justify-between">
+                        <dt className="text-sm font-medium text-muted-foreground">Auto-add root DSE:</dt>
+                        <dd className="text-sm">
+                          {formData.autoAddRootDSE ? 'Yes' : 'No'}
+                        </dd>
+                      </div>
                       <div className="flex justify-between">
                         <dt className="text-sm font-medium text-muted-foreground">Create groups per attribute value:</dt>
                         <dd className="text-sm">