Skip to content

Wrong Parameter Name when creating an AppRole#8818

Closed
mikail-gultekin wants to merge 1 commit into
hashicorp:mainfrom
mikail-gultekin:patch-1
Closed

Wrong Parameter Name when creating an AppRole#8818
mikail-gultekin wants to merge 1 commit into
hashicorp:mainfrom
mikail-gultekin:patch-1

Conversation

@mikail-gultekin
Copy link
Copy Markdown

"token_policies" parameter does not work when creating an AppRole. If you use this parameter name, you are not able to create an AppRole attached to the policy that you have specified along with this parameter. Instead, I've changed this parameter name to "policies" and it works. So, the documentation was not up to date, I've changed this info so users will not be confused.

"token_policies" parameter does not work when creating an AppRole. If you use this parameter name, you are not able to create an AppRole attached to the policy that you have specified along with this parameter. Instead, I've changed this parameter name to "policies" and it works. So, the documentation was not up to date, I've changed this info so users will not be confused.
@hashicorp-cla
Copy link
Copy Markdown

hashicorp-cla commented Apr 23, 2020

CLA assistant check
All committers have signed the CLA.

@ncabatoff
Copy link
Copy Markdown
Collaborator

Hi @mikail-gultekin,

Thank you for the PR!

I suspect you're using an older Vault version than the latest, which is what the website documentation targets. policies has been deprecated in favour of token_policies in current Vault versions.

I think it's a good idea to document the deprecated option too, though it should be flagged as such and a note added to recommend the newer option that replaces it. I don't think it should appear in the example however, so please revert that part.

@heatherezell
Copy link
Copy Markdown
Contributor

Hi @mikail-gultekin,

Thank you for the PR!

I suspect you're using an older Vault version than the latest, which is what the website documentation targets. policies has been deprecated in favour of token_policies in current Vault versions.

I think it's a good idea to document the deprecated option too, though it should be flagged as such and a note added to recommend the newer option that replaces it. I don't think it should appear in the example however, so please revert that part.

Hi folks! I'm going to go ahead and close this PR - please feel free to re-open it (and rebase off of main) if the questions can be answered and you'd like to continue working on it. Thanks!

eualin pushed a commit to eualin/vault that referenced this pull request Sep 23, 2025
…shicorp#8818)

Co-authored-by: John-Michael Faircloth <fairclothjm@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.

5 participants