Skip to content

docs: Note rejected SDK key retry in the initialization matrix row - #68

Draft
kinyoklion wants to merge 1 commit into
mainfrom
devin/1791384372-readme-rejected-key-init
Draft

kinyoklion wants to merge 1 commit into
mainfrom
devin/1791384372-readme-rejected-key-init

Conversation

@kinyoklion

Copy link
Copy Markdown
Member

Summary

LaunchDarkly Python SDK 9.18.0 made a rejected SDK key retryable rather than permanently stopping the data source. The README says start_wait=0 waits "until the data source becomes valid or permanently fails", but a rejected key now never permanently fails. With zero, initialize therefore never completes while the key is rejected.

  • Initialization row: now ⚠️, notes the SDK 9.18.0 behavior, and marks the provider gap (zero should mean "do not wait", per OFP 4.3.6), tracked by #64.
  • The start_wait paragraph gets the same note.
Details

I checked this locally with launchdarkly-server-sdk 9.18.1 and an invalid key. The data source stayed INITIALIZING, and last_error reported ERROR_RESPONSE 401 while it retried. It never reported OFF.

The provider allows launchdarkly-server-sdk >=9.17.0,<10, so new installs get the retry behavior. #64 also edits these lines. Whichever merges second will need a small README conflict fix.

Found in the weekly OpenFeature provider audit. Docs-only change.

Link to Devin session: https://app.devin.ai/sessions/c8bc4e80339b473882c2dddffe981edf
Open in Devin Desktop: https://app.devin.ai/desktop/session/c8bc4e80339b473882c2dddffe981edf?variant=devin
Requested by: @kinyoklion

Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant