Skip to content

Add option to disable LDAPS Certificate Validation - #1096

Merged
ssddanbrown merged 3 commits into
BookStackApp:masterfrom
christophert:add-ldaptlsinsecure
Dec 22, 2018
Merged

ssddanbrown merged 3 commits into
BookStackApp:masterfrom
christophert:add-ldaptlsinsecure

Conversation

@christophert

Copy link
Copy Markdown
Contributor

This PR allows users to set LDAP_TLS_INSECURE in their .env to bypass LDAP TLS Certificate Verification. This is useful for development environments that may not have a trusted CA certificate for LDAP functionality.

@ssddanbrown

Copy link
Copy Markdown
Member

@christophert Thanks for offering this pull request. Have set for next feature release.

Will require a minor update to the required PHP version for BookStack due to the LDAP_OPT_X_TLS_REQUIRE_CERT constant only being available in 7.0.5 and above as per details here.

@lucassmacedo

Copy link
Copy Markdown

Nice!!

@christophert

Copy link
Copy Markdown
Contributor Author

Docs update PR submitted as well: BookStackApp/website#32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants