Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

credstash-grant-reader could not be planned with empty(default) context_keys and context_values #265

Open
Magicloud opened this issue Oct 10, 2019 · 2 comments · May be fixed by #266
Open
Assignees

Comments

@Magicloud
Copy link
Contributor

Checking out the code, this line looks weird to me.

element(var.context_keys, count.index) = element(var.context_values, count.index)

Seems like I must pass one and only one key/value pair for each role. Wondering if this code was not as it supposed to be.

@Magicloud Magicloud linked a pull request Oct 11, 2019 that will close this issue
@ketzacoatl
Copy link
Contributor

@Magicloud can you clarify: how can we help, or what would you like to see done?

@Magicloud
Copy link
Contributor Author

@ketzacoatl This was the confusing code piece I chatted with you the other day. I think it is a mis-implementation. Thus I have that PR. Please review.

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 a pull request may close this issue.

4 participants