Skip to content

Conversation

LaunchDarklyReleaseBot
Copy link
Contributor

@LaunchDarklyReleaseBot LaunchDarklyReleaseBot commented Jul 15, 2021

[6.2.1] - 2021-07-15

Changed:

  • If variation or variation_detail is called with a user object that has no key (an invalid condition that will always result in the default value being returned), the SDK now logs a warn-level message to alert you to this incorrect usage. This makes the Ruby SDK's logging behavior consistent with the other server-side LaunchDarkly SDKs. (#177)

eli-darkly and others added 26 commits January 21, 2021 10:56
revert renames of feature_store & update_processor
…l-user

don't send event for nil user evaluation
* WIP - from sam's pairing session

* starting sdk changes

* adding tests and making sure everything works

* adding more tests

* removing the singleton for fallthrough

* Revert "removing the singleton for fallthrough"

This reverts commit dff7adbb809ecc63118d0fbff9742a88a039c679.

* taking a different approach to keep things immutable

* adding tests for untracked

* remove unnecessary comment

* making sure to return two values in all code paths

Co-authored-by: pellyg-ld <gpelly@launchdarkly.com>
The in_experiment attribute was added to reasons as part of #150 but it doesn't appear to be received in events. I think that's because it's sending it in JSON as "in_experiment" rather than "inExperiment" as we expect to parse it.
* fixing ruby logic

* adding missing spec

* Apply suggestions from code review

Co-authored-by: Sam Stokes <sstokes@launchdarkly.com>

* pr tweaks

* making spec language consistent

Co-authored-by: Sam Stokes <sstokes@launchdarkly.com>
* add log warnings for nil/empty user key

* rm warning for empty string key

* fix test
@eli-darkly eli-darkly requested a review from bwoskow-ld July 15, 2021 20:30
@LaunchDarklyReleaseBot LaunchDarklyReleaseBot merged commit 328b2c8 into master Jul 15, 2021
@LaunchDarklyReleaseBot LaunchDarklyReleaseBot deleted the release-6.2.1 branch July 15, 2021 20:31
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.

9 participants