Skip to content
This repository was archived by the owner on May 14, 2024. It is now read-only.

Server - Consistent BindDN #750

Merged
merged 6 commits into from
Aug 3, 2021
Merged

Server - Consistent BindDN #750

merged 6 commits into from
Aug 3, 2021

Conversation

tpretz
Copy link
Contributor

@tpretz tpretz commented Jul 12, 2021

Ensuring a connection in an anonymous state has a consistent bindDN variable, regardless of being implicit, or explicitly bound.

Added hook to switch to anonymous on unbind too.
Server should close such connections but added as a safeguard to ensure permissions are dropped regardless of connection termination state.

#748

tpretz added 3 commits July 12, 2021 16:02

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
@UziTech UziTech linked an issue Jul 14, 2021 that may be closed by this pull request
Copy link
Member

@UziTech UziTech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@jsumners jsumners left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

UziTech added 3 commits August 3, 2021 10:07

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@UziTech UziTech merged commit fb6816d into ldapjs:master Aug 3, 2021
@UziTech UziTech mentioned this pull request Aug 3, 2021
@jsumners
Copy link
Member

⚠️ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.

Please include a minimal reproducible example
when opening a new issue.

@ldapjs ldapjs locked as resolved and limited conversation to collaborators Mar 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Anonymous Bind DN Inconsistencies
3 participants