-
Notifications
You must be signed in to change notification settings - Fork 18
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
chore: merge private to public #105
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
update for go-jsonstream API changes and use latest package releases
build in Windows with Go 1.14
bump go-sdk-common version to get user JSON parsing fix
…-chars fail fast if SDK key has invalid characters, don't log the key
remove excessive logging in ldfilewatch, add log message for reloading file
bump go-server-sdk-evaluation to get semver parsing fix
…ta-logging remove spurious logging of stream data
update to go-server-sdk-evaluation 1.1.2 for ch100737 fix
…#288) * add unbounded segments interfaces, status provider, test suite * lint * add package comment * better component organization * rename cache time to poll interval, improve tests * rename segment keys to segment refs in unbounded segments API * fix naming some more
…lease packages (#293)
# Conflicts: # CHANGELOG.md # internal/datasource/requestor.go # internal/datasource/streaming_data_source.go # internal/version.go # testhelpers/storetest/big_segment_store_test_suite.go # testhelpers/storetest/big_segment_store_test_suite_test.go
# Conflicts: # CHANGELOG.md
… always-true conditional (#361) The persistentDataStoreWrapper's Init method is concerned with initializing its underlying store and then setting up the caching layer. The conditional check at the end of the function that guards setting the initialization status is not necessary, because it's not possible to arrive at the check without either (no error) or (cache is infinite ttl) being true.
…test releases * Update to real go-sdk-common v3.0.1 release * Update go-sdk-events to v2.0.1 release * Update go-server-sdk-evaluation to v2.0.2 release The go-sdk-common patch fixes an umarshaling bug in easyJSON context deserializer.
Merge the changelog change back to private
* Add Payload Filter configuration option for streaming and polling data sources Allows the SDK to specify a filter key, which causes a subset of data to be sent over the streaming connection based on a filter configured in LaunchDarkly. * Update polling tests * Update streaming tests * Add capability 'filtering' to contract test service * Add mock_polling_request.go to mocks package
Co-authored-by: Louis Chan <[email protected]>
A 413 is considered a locally unrecoverable error. It should not prevent subsequent events from being sent. This was fixed upstream in the events package, so we need only bump a version here.
Switches over from CircleCI to github actions.
This pull request has been linked to Shortcut Story #234118: Switch to github actions. |
louis-launchdarkly
approved these changes
Feb 23, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.