NuKVM/xen-orchestra
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(xo-server-auth-ldap): typo defaults → default (#5388)

Browse Source 
The "Check certificate" option's default value was incorrectly set in the UI. So if it was left untouched by the user (showing as OFF since `undefined`), it would be saved as `undefined`, and then fallback to the actual default value (`true`) when used in the code. So it would try to check the certificate even though the user didn't want to.
This commit is contained in:
Pierre Donias 2020-11-20 14:10:58 +01:00 committed by GitHub
parent ee7217c7c9
commit e0987059d3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CHANGELOG.unreleased.md
View File

@ -37,6 +37,7 @@
> >
> In case of conflict, the highest (lowest in previous list) `$version` wins. > In case of conflict, the highest (lowest in previous list) `$version` wins.
- xo-server-auth-ldap patch
- @vates/multi-key-map minor - @vates/multi-key-map minor
- @xen-orchestra/fs patch - @xen-orchestra/fs patch
- xo-server minor - xo-server minor

2
packages/xo-server-auth-ldap/src/index.js
View File

@ -53,7 +53,7 @@ If not specified, it will use a default set of well-known CAs.
description: description:
"Enforce the validity of the server's certificates. You can disable it when connecting to servers that use a self-signed certificate.", "Enforce the validity of the server's certificates. You can disable it when connecting to servers that use a self-signed certificate.",
type: 'boolean', type: 'boolean',
defaults: DEFAULTS.checkCertificate, default: DEFAULTS.checkCertificate,
}, },
startTls: { startTls: {
title: 'Use StartTLS', title: 'Use StartTLS',