Add optional support for passing env variables into provider#121
Add optional support for passing env variables into provider#121kuzaxak wants to merge 1 commit intocarlpett:masterfrom
Conversation
Needed to support dynamic auth as described in [issue][1], as we are using sops dependency which is using native aws sdk, it will read such configuration from env variables of the process. Additionally, I upgraded deprecated terraform SDK such as replacing `Read` into `ReadContext` and added [1]: carlpett#112
|
@carlpett hey, could you please have a look? |
|
Hi @kuzaxak, |
That's amazing! Hope you have a great time :)
@carlpett Sure, will do. Could you give me a hint what kind of issues you faced? |
|
Ooops, apparently didn't press the "Comment" button, so this has been sitting in a tab a week 🤦 Thank you for the well wishes! :) I sadly don't remember what didn't work, as it's been a few years. I can't find any old issues that would indicate I actually made a release with it, so it was probably something quite obvious. Tests or even compilation failing, or something like that. |
|
Hello @kuzaxak any chance that you intend to finish the work on this PR? 😇 |
|
This PR would be very useful for us, too. |
To support dynamic auth as described in carlpett#112, this commit adds a new `env` attribute to allow users to pass environment variables to provider. Resolves: - carlpett#112 - carlpett#121 Signed-off-by: Sunghoon Kang <sunghoon.kang@trillionlabs.co>
|
Just wanted to share that I’ve been exploring a similar idea in a separate implementation — focusing on handling You can see the concept, practical examples, and details on which problems this approach tries to solve here: Might be useful for comparison or future design discussions. |
Needed to support dynamic auth as described in issue, as we are using sops dependency which is using native aws sdk, it will read such configuration from env variables of the process.
Additionally, I upgraded deprecated terraform SDK such as replacing
ReadintoReadContextand added