Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Adding OpenFeature provider for AWS AppConfig #310

Open
wants to merge 44 commits into
base: main
Choose a base branch
from

Conversation

wani-guanxi
Copy link

This PR

Adding new project for AWS AppConfig provider implementation for OpenFeature. This allows to manage feature flags using AWS AppConfig.

  • Provider for AWS AppConfig

Notes

Additional details about the package is in Readme file.

Follow-up Tasks

  • This implementation currently does not support multi-variant AppConfig Feature flags. Or rather there is no way to pass on calling context to the request to AWS AppConfig. I am looking at documentation to figure out how this is done, but haven't got much far on that. Will be looking to add that soon.

How to test

Testing instructions are available in Readme file under Setup section. Thing to note, this will require AWS account to generate access key and ID in order to talk to AWS AppConfig.

@wani-guanxi wani-guanxi requested review from a team as code owners January 18, 2025 15:32
@wani-guanxi wani-guanxi changed the title Feature/appconfig provider feat: Adding OpenFeature provider for AWS AppConfig Jan 18, 2025
@wani-guanxi wani-guanxi force-pushed the feature/appconfig-provider branch from 86985c4 to bf18ec7 Compare January 18, 2025 16:23
…feature#302)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Ash.Wani <[email protected]>
@beeme1mr
Copy link
Member

Hey @wani-guanxi, sorry for the delay. I'll try and provide feedback in the next few days.

Signed-off-by: Ash.Wani <[email protected]>
…uteValue format, as supported by AWS app config.

Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
…o be supplied at the time of setup

Signed-off-by: Ash.Wani <[email protected]>
Signed-off-by: Ash.Wani <[email protected]>
Missed updating comments over the course of changes. Fixing that.

Signed-off-by: Ash.Wani <[email protected]>
@wani-guanxi wani-guanxi force-pushed the feature/appconfig-provider branch from ff6a6ae to c31db95 Compare January 27, 2025 22:55
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.

3 participants