Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

prepare 2.17.2 release #206

Merged
merged 525 commits into from
Mar 18, 2020
Merged

prepare 2.17.2 release #206

merged 525 commits into from
Mar 18, 2020

Conversation

LaunchDarklyCI
Copy link
Contributor

[2.17.2] - 2020-03-18

Fixed:

  • Some users reported an error where the SDK said that the content type of a response was application/json, application/json; charset=utf8. It is invalid to have multiple Content-Type values in a response and the LaunchDarkly service does not do this, but an improperly configured proxy/gateway might add such a header. Now the SDK will tolerate a value like this as long as it starts with application/json. (#205)
  • Fixed incorrect usage of Object.hasOwnProperty which could have caused an error if a feature flag had hasOwnProperty as its flag key.

eli-darkly and others added 30 commits March 11, 2019 13:18
pin Node version to 11.10.1 due to Jest problem
add support for optional trackReasons flag for experimentation
replace rm with rimraf in react wrapper build
coerce user attributes into strings when necessary
asyncify tests, simplify HTTP mocking, general cleanup
LaunchDarklyCI and others added 26 commits January 27, 2020 21:21
use js-sdk-common 3.1.2 and loosen our other dependency
# Conflicts:
#	package-lock.json
#	package.json
update js-sdk-common for content type bugfix; update some dev dependencies
@eli-darkly eli-darkly requested a review from bwoskow-ld March 18, 2020 20:22
@LaunchDarklyCI LaunchDarklyCI merged commit 5b550be into master Mar 18, 2020
@LaunchDarklyCI LaunchDarklyCI deleted the release-2.17.2 branch March 18, 2020 21:28
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.

7 participants