open-feature / open-feature/ruby-sdk
Supply the bound domain to provider initialization
Nobody has claimed this yet.
- Dominant language
- Ruby
- Stars
- 43
- Forks
- 18
- Avg merge
- 6h 21m
- Merged PRs (30d)
- 19
Description
The spec now supplies the bound domain to a provider's initialize function, and adds an opt-in domain-scoped provider declaration that the API enforces by binding such a provider to at most one domain. This lets stateful providers (e.g. an OFREP static-context provider with a persistent cache) scope their state to a single, unambiguous domain.
What needs to happen
- Pass the bound
domain(if any) to the provider'sinitwhen it's registered via the provider mutator - Extend the provider
initto accept the optionaldomain(non-breaking, additional optional parameter) - Add a way for a provider to declare itself
domain-scoped - Enforce that a
domain-scopedprovider is bound to at most onedomain, rejecting any attempt to bind an already-bound instance to anotherdomain - Associated testing and docs
Links
- Parent issue: https://github.com/open-feature/spec/issues/403
- Spec sections: 1.1 Flag evaluation, 2.4 Initialization
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the provider mutator and provider init entry points described in the issue, then trace how provider declarations are registered and bound to a domain. Add the optional domain argument and domain-scoped binding enforcement, with associated tests and documentation. Done means domain-scoped providers cannot bind to multiple domains while existing providers remain compatible.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ruby
- Domain
- api
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100