revert(ldap): remove CTL-specific AD connection prefill
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:
@@ -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">
|
||||||
|
|||||||
@@ -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",
|
||||||
|
|||||||
@@ -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",
|
||||||
|
|||||||
Reference in New Issue
Block a user