revert(ldap): remove CTL-specific AD connection prefill
Tessera CI/CD / Lint & Type Check (push) Successful in 42s
Tessera CI/CD / Tests (push) Successful in 46s
Tessera CI/CD / Build & Publish Images (push) Successful in 1m27s

The prior quick task (260707-csw) pre-filled the LDAP connection form
with one customer's specific Active Directory values (balios.ctl.local,
dc=ctl,dc=local) and a matching bind-DN hint. User clarified this was
never wanted -- Tessera is a generic multi-tenant product, and baking
one customer's infrastructure into the shared admin UI is wrong,
especially since a `dcdown -v` reinstall surfaced it unexpectedly as
seemingly-baked-in defaults on a fresh system.

Reverted to blank fields with generic example placeholders
(ldap.example.com / dc=example,dc=com / cn=admin,dc=example,dc=com),
matching the form's state before that change. Removed the now-unused
bindDnHint i18n key.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
2026-07-08 09:33:34 +02:00
parent 2ad875b2cd
commit 8e8305ce18
3 changed files with 4 additions and 12 deletions
+4 -10
View File
@@ -61,17 +61,12 @@ export default function AdminLdapPage() {
const [syncing, setSyncing] = useState(false); const [syncing, setSyncing] = useState(false);
// Form state for connection settings. // Form state for connection settings.
// Defaults are pre-filled with the known-good CTL Active Directory
// connection values (sourced from the working XWiki LDAP config) so a
// brand-new setup only needs the service-account password. fetchConfig()
// below overwrites these with the real saved values whenever a config
// already exists.
const [formData, setFormData] = useState({ const [formData, setFormData] = useState({
serverUrl: 'ldap://balios.ctl.local:3268', serverUrl: '',
baseDn: 'dc=ctl,dc=local', baseDn: '',
bindDn: '', bindDn: '',
bindPassword: '', bindPassword: '',
searchFilter: '(&(objectClass=user)(objectCategory=person))', searchFilter: '(objectClass=person)',
syncIntervalMin: 60, syncIntervalMin: 60,
isActive: true, isActive: true,
}); });
@@ -342,11 +337,10 @@ export default function AdminLdapPage() {
type="text" type="text"
value={formData.bindDn} value={formData.bindDn}
onChange={(e) => setFormData({ ...formData, bindDn: e.target.value })} onChange={(e) => setFormData({ ...formData, bindDn: e.target.value })}
placeholder="ctl\serviceaccount" placeholder="cn=admin,dc=example,dc=com"
className="flex h-10 w-full rounded-md border border-input bg-background px-3 py-2 text-sm" className="flex h-10 w-full rounded-md border border-input bg-background px-3 py-2 text-sm"
required required
/> />
<p className="text-xs text-muted-foreground">{t('bindDnHint')}</p>
</div> </div>
<div className="space-y-2"> <div className="space-y-2">
<label className="text-sm font-medium text-foreground"> <label className="text-sm font-medium text-foreground">
-1
View File
@@ -293,7 +293,6 @@
"testFailed": "Verbindung fehlgeschlagen", "testFailed": "Verbindung fehlgeschlagen",
"enable": "Aktiviert", "enable": "Aktiviert",
"disable": "Deaktiviert", "disable": "Deaktiviert",
"bindDnHint": "AD-Format: Domaene\\Benutzername (z.B. ctl\\serviceaccount), keine DN-Schreibweise.",
"fieldMapping": { "fieldMapping": {
"title": "Feld-Zuordnung", "title": "Feld-Zuordnung",
"add": "Zuordnung hinzufuegen", "add": "Zuordnung hinzufuegen",
-1
View File
@@ -293,7 +293,6 @@
"testFailed": "Connection failed", "testFailed": "Connection failed",
"enable": "Enabled", "enable": "Enabled",
"disable": "Disabled", "disable": "Disabled",
"bindDnHint": "AD format: domain\\username (e.g. ctl\\serviceaccount), not DN notation.",
"fieldMapping": { "fieldMapping": {
"title": "Field mapping", "title": "Field mapping",
"add": "Add mapping", "add": "Add mapping",