Skip to content

[flagd] Fix in-process provider to use FLAGD_SYNC_PORT (not FLAGD_PORT) for port configuration (backwards compatibility) #318

@aepfli

Description

@aepfli

Background

Recent updates to flagd and related tests in flagd-testbed clarified the use of environment variables for ports:

  • FLAGD_PORT is for remote/server mode.
  • FLAGD_SYNC_PORT should be used for in-process/sync providers.

Currently, the in-process provider uses the wrong environment variable (FLAGD_PORT).

Required Changes

  • Fix provider configuration logic to use FLAGD_SYNC_PORT for in-process provider setups, and fall back to FLAGD_PORT for backwards compatibility.
  • Update any relevant documentation and configuration references.
  • Ensure test coverage, ideally using scenarios matching those in flagd-testbed (with @legacy-sync-port tag for backwards compatibility coverage).

References

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions