Skip to content

feat(flags): switch local evaluation endpoint to /flags/definitions#179

Closed
patricio-posthog wants to merge 4 commits into
masterfrom
feat/flags-definitions-endpoint
Closed

feat(flags): switch local evaluation endpoint to /flags/definitions#179
patricio-posthog wants to merge 4 commits into
masterfrom
feat/flags-definitions-endpoint

Conversation

@patricio-posthog
Copy link
Copy Markdown
Contributor

Motivation and Context

The Rust feature flags definitions fleet now serves 100% of /api/feature_flag/local_evaluation traffic in all environments. This switches the SDK's default polling URL from the legacy Django path to the Rust endpoint's native path (/flags/definitions).

The old /api/feature_flag/local_evaluation path remains registered as a route alias on the Rust service, so older SDK versions continue to work.

How did you test it?

  • Updated all test mock HTTP servers to match the new path
  • go build ./... passes

@patricio-posthog patricio-posthog requested a review from a team as a code owner April 15, 2026 16:47
@patricio-posthog patricio-posthog added release Add this label to trigger a release from a PR bump-patch Trigger a patch bump when merging this PR labels Apr 15, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 15, 2026

posthog-go Compliance Report

Date: 2026-04-15 17:33:25 UTC
Duration: 219ms

⚠️ Some Tests Failed

0/1 tests passed, 1 failed


Feature_Flags Tests

⚠️ 0/1 tests passed, 1 failed

View Details
Test Status Duration
Request Payload.Request With Person Properties Device Id 5ms

Failures

request_payload.request_with_person_properties_device_id

404, message='Not Found', url='http://sdk-adapter:8080/get_feature_flag'

@patricio-posthog patricio-posthog force-pushed the feat/flags-definitions-endpoint branch from f9ea871 to 7c38b6e Compare April 15, 2026 17:21
Copy link
Copy Markdown
Contributor

@dustinbyrne dustinbyrne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump-patch Trigger a patch bump when merging this PR release Add this label to trigger a release from a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants