open-feature / spec

OpenFeature specification
https://openfeature.dev
Apache License 2.0
612 stars 35 forks source link

No client/provider instances, add metadata #93

Closed toddbaert closed 2 years ago

toddbaert commented 2 years ago

This replaces the instances of client and provider from the hookContext, instead adding metadata fields which contain name props. I chose a metadata field because it gives us some flexibility to add more such properties later

Closes: https://github.com/open-feature/spec/issues/84 Closes: https://github.com/open-feature/spec/issues/85

Let me know what you guys think.

@davejohnston This relates to a concern you brought up in slack: https://cloud-native.slack.com/archives/C0344AANLA1/p1653426865053149

beeme1mr commented 2 years ago

Looks good to me. FYI, @justinabrahms.