| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Sorry, something went wrong.
Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
Co-Authored-By: rlamb@launchdarkly.com <4955475+kinyoklion@users.noreply.github.com>
| Back | FazBrowse Home | New Git URL |
Adds a start_wait parameter to LaunchDarklyProvider, defaulting to the LaunchDarkly SDK's five seconds.
- The value is passed to LDClient(config, start_wait) and bounds the whole of initialization: with a positive value initialize completes with the outcome the constructor already waited for, rather than waiting again, so a 5 second start wait cannot become a 10 second wait for set_provider_and_wait.
- Initialization fails when the client did not become ready in time; the provider keeps reporting status afterward, so a later connection still makes it ready.
- Zero does not block the constructor at all, and initialize then waits without a deadline.
Implementation detailsTesting: make test, make lint. Tests cover the default matching the SDK default and initialization failing without waiting a second time.
Related spec change: https://github.com/launchdarkly/sdk-specs/pull/257
Link to Devin session: https://app.devin.ai/sessions/0c452d209ec54b068ba120b4c92b8f6c
Open in Devin Desktop: https://app.devin.ai/desktop/session/0c452d209ec54b068ba120b4c92b8f6c?variant=devin
Requested by: @kinyoklion