Skip to content

Add docs for OpenLDAP plugin's new AD schema#9619

Merged
tomhjp merged 3 commits into
masterfrom
openldap-plugin-ad-schema-docs
Aug 10, 2020
Merged

Add docs for OpenLDAP plugin's new AD schema#9619
tomhjp merged 3 commits into
masterfrom
openldap-plugin-ad-schema-docs

Conversation

@tomhjp
Copy link
Copy Markdown
Contributor

@tomhjp tomhjp commented Jul 29, 2020

This PR adds documentation for a new "ad" schema for the OpenLDAP plugin. This is to support rotating passwords using the unicodePwd field instead of userPassword.

See hashicorp/vault-plugin-secrets-openldap#16 for the implementation.

@hashicorp-cla
Copy link
Copy Markdown

hashicorp-cla commented Jul 29, 2020

CLA assistant check
All committers have signed the CLA.

@tomhjp tomhjp force-pushed the openldap-plugin-ad-schema-docs branch from b7c62a3 to 157befd Compare July 31, 2020 09:36
@tomhjp tomhjp requested a review from jasonodonnell July 31, 2020 09:37
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Copy link
Copy Markdown
Contributor

@jasonodonnell jasonodonnell left a comment

Choose a reason for hiding this comment

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

LGTM

@tomhjp tomhjp merged commit d3f43d6 into master Aug 10, 2020
@tomhjp tomhjp deleted the openldap-plugin-ad-schema-docs branch August 10, 2020 09:24
tomhjp added a commit that referenced this pull request Aug 17, 2020
* Backport #9673: Update OpenLDAP secrets plugin 0.1.4 -> 0.1.5

* Add docs for OpenLDAP plugin's new AD schema (#9619)
pull Bot pushed a commit to sigtrap/vault that referenced this pull request Sep 26, 2025
) (hashicorp#9619)

* copy changes WIP

* update descriptions and headers to match latest designs

* add details to method info table

* update test assertion

Co-authored-by: claire bontempo <68122737+hellobontempo@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants