Skip to content

[FEATURE] update ACL Token Read endpoint to add expnded parameter#601

Open
MohamedM216 wants to merge 2 commits intoG-Research:masterfrom
MohamedM216:acl-token-read
Open

[FEATURE] update ACL Token Read endpoint to add expnded parameter#601
MohamedM216 wants to merge 2 commits intoG-Research:masterfrom
MohamedM216:acl-token-read

Conversation

@MohamedM216
Copy link
Contributor

@MohamedM216 MohamedM216 commented Mar 10, 2026

Description

update ACL Token Read endpoint to add expnded parameter

Related Issues

#531

Additional Context

Checklist

Please make sure to check the following before submitting your pull request:

  • Did you read the contributing guidelines?
  • Did you update the docs?
  • Did you write any tests to validate this change?

Copy link
Contributor

@marcin-krystianc marcin-krystianc left a comment

Choose a reason for hiding this comment

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

HI @MohamedM216,

look at the example of a response when the expanded=true (https://developer.hashicorp.com/consul/api-docs/acl/tokens#sample-response-1). See, there are two new fields, ExpandedPolicies and ExpandedRoles.
The test should assert that these fields are filled in, when expanded=true and null when expanded=false.

@MohamedM216
Copy link
Contributor Author

ExpandedPolicies and ExpandedRoles don't exist in QueryResult<TokenEntry> or PolicyLink. PolicyLink only has ID and Name. Any ideas?

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