fix(secrets): allow dynamic secrets to be read#398
fix(secrets): allow dynamic secrets to be read#398ibacalu wants to merge 1 commit intohashicorp:mainfrom
Conversation
This would allow reading of `data.*` thus accessing all dynamic secret properties Resolves hashicorp#396
|
Gentle bump, could you please review this? 🙏🏼 |
|
Hi, would you say this review accomplishes the same objective of allowing users to read all properties of a secret at once? If yes, I think it'd be preferable to focus on #238 as I find the added functionality of being able to optionally add a prefix useful to avoid collisions and group related secret properties. |
|
@maxcoulombe I haven't tested out #238. It could work as well although the actual use case might be different. secrets: |
postgres/creds/dbname * | DB_CRED;The trick with dynamic secrets is that you can only make 1 request. Every additional request is going to generate different values. |
|
Closing as this should be fixed by #488. Please feel free to reopen if you feel this was a mistake, or if there is additional information to add. Thanks! |
This would allow reading of
data.*thus accessing all dynamic secret properties.Resolves #396