Skip to content

Conversation

@grantorchard
Copy link
Contributor

Hey team, I discovered ldaps requirement while going through the AD backend configuration in a customer's lab environment. Not sure if it's worth adding here, or mentioning at the configuration example.

Copy link
Contributor

@tyrannosaurus-becks tyrannosaurus-becks left a comment

Choose a reason for hiding this comment

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

Hey, thanks for adding this! Much appreciated!

@tyrannosaurus-becks tyrannosaurus-becks merged commit b33bf64 into master Apr 30, 2020
@tyrannosaurus-becks tyrannosaurus-becks deleted the grantorchard-patch-1 branch April 30, 2020 16:26
@tyrannosaurus-becks tyrannosaurus-becks restored the grantorchard-patch-1 branch April 30, 2020 16:26
sapk pushed a commit to sapk-fork/vault that referenced this pull request Aug 27, 2025
… definitions into main into release/1.20.x+ent into ce/release/1.20.x (hashicorp#8917)

* VAULT-38888 Add prefix vault to metric summary definitions into main (hashicorp#8725) (hashicorp#8895)

* VAULT-38888 Add prefix vault to metric summary definitions

* VAULT-38888 Add changelog for fix

* Edit changelog file name

---------

Co-authored-by: roh-ag <[email protected]>

* Remove ent. file

---------

Co-authored-by: roh-ag <[email protected]>
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.

2 participants