karlgw / azcfg Goto Github PK

There might be situations where missing secrets might be fine. Add an option and/or tag addition for ignoring secrets that cannot be retrieved.

Add examples for various of the functions that are provided by the package.

Due to not needing all the features provided by the azidentity module, it brings a lot of dependencies into the project and users of this module. Instead use https://github.com/AzureAD/microsoft-authentication-library-for-go.

This will require some refactoring on internal workings and the interfaces on what clients that can be used.

Since in theory a custom made client implementing the KeyVaultClient can be used to handle other cloud providers implementations of "vaults" the name VaultClient would be more appropriate for the interface.

Options can be set on package level but not for single calls to Parse. Address this.

Currently parsing supports slices but not slices of pointers.

Investigate on sensible errors when getting errors when fetching secrets.

For certain scenarios it would be beneficial to pass a configured "Parser" down into parts of an application for further use instead of calling the package directly.

Suggested:

Add a Parser type, which has the method Parse. Should be created with package function NewParser.

This module only uses a small subset of the azsecret module, thus a custom implementation is in order.

Due to the default behaviour of azidentity.NewDefaultAzureCredential and the environment variable it looks for, this module should rely on another set of environment variable to allow for them to work together side by side better.

Suggestion:

• Add prefix AZCFG_ to all environment variables that has to do with this module.

This will require an update that does not make use of azidentity.NewDefaultAzureCredential.

Add support for uint:

• uint
• uint8
• uint16
• uint32
• uint64