Closed benjiro closed 1 year ago
Let's continue the discussion about the following scenario described by @kinyoklion here https://github.com/open-feature/dotnet-sdk/pull/129#discussion_r1210810970
cc @lopitz @lukas-reining
@benjiro FYI I moved this from the dotnet-sdk repo.
I totally agree. Also found that and implemented it in the JS SDK the way that this does not happen for the case between named and unnamed providers. But scenario 2 is not handled by the JS SDK now, I will fix that @toddbaert.
I totally agree. Also found that and implemented it in the JS SDK the way that this does not happen for the case between named and unnamed providers. But scenario 2 is not handled by the JS SDK now, I will fix that @toddbaert.
Let's wait for now to build some consensus, just so that you dont implement something that you have to change. Do you agree with the solution I've vaguely proposed?
Any alternatives?
Yes I totally agree with that @toddbaert, this is fixing at least the shutdown for me. I already did that, was just 3 lines of code and a test. Will add the PR and we can merge it or change it once we have concencus what do you think?
Yes I totally agree with that @toddbaert, this is fixing at least the shutdown for me. I already did that, was just 3 lines of code and a test. Will add the PR and we can merge it or change it once we have concencus what do you think?
For now maybe add the PR here for reference.
I fixed that in the JS SDK now. I had this check for the default provider already, but missed to do it for several named clients. https://github.com/open-feature/js-sdk/pull/444
Thanks Todd. I agree with what's been stated:
basic:
2 names:
2 names 2:
I think this is taken care of with https://github.com/open-feature/spec/pull/193. Please re-open if you disagree, @benjiro
Spec: open-feature/spec@a4ffec3 See: https://github.com/open-feature/dotnet-sdk/issues/126
The spec allows a single instance of a provider to be bound to multiple names (even if it didn't allow this, any naive implementation would probably allow it to happen). We either need to prevent it or specify how it should work. We may want to specify that providers that are still bound to any client name are not shut down. - @toddbaert
I consider this a spec bug personally, because it constitutes a problematic ambiguity in the spec. - @toddbaert
cc @justinabrahms
Quote from @kinyoklion :+1: