-
Notifications
You must be signed in to change notification settings - Fork 19
Closed
Description
[flagd] use sync_context
field instead of GetMetadata
Background
Currently, to support static context enrichment via attributes sent from flagd, in-process providers call sync.GetMetadata along with starting the stream. This is not ideal. The stream isn't valid without the metadata response (since it could result in rules not functioning as expected due to missing attributes) and the metadata response is useless without the stream.
This has been implemented in flagd with this PR.
We need to absorb this change by making use of this new field in in-process providers
Requirements
- while initializing the
SyncFlags
stream, continue to callgetMetadata
, but override it with the results from thesync_context
field, if present (it's optional, so its presence can be checked). - associated tests
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
Done