open-feature / spec

OpenFeature specification
https://openfeature.dev
Apache License 2.0
664 stars 34 forks source link

Requirement 2.5.2 might not be relevant anymore #270

Open fabriziodemaria opened 3 weeks ago

fabriziodemaria commented 3 weeks ago

Link to requirement here. It reads:

After a provider's shutdown function has terminated successfully, the provider's state MUST revert to its uninitialized state. If a provider requires initialization, once it's shut down, it must transition to its initial NOT_READY state. Some providers may allow reinitialization from this state. Providers not requiring initialization are assumed to be ready at all times.

According to recent spec changes, the Provider Status is now meant to be maintained by the SDK, so it's not a Provider's responsibility to transition to NOT_READY state.

lukas-reining commented 3 weeks ago

If a provider requires initialization, once it's shut down, it must transition to its initial NOT_READY state. Some providers may allow reinitialization from this state. Providers not requiring initialization are assumed to be ready at all times.

Mh, I think the content here is right but has to be adjusted to "the clients status accessor" instead of "providers status". But it is a good point, I think we should adjust this and move it to section one. Does that make sense?