Skip to content

fix: improves handling of invalid contexts and adds SSE Client logging.#207

Merged
tanderson-ld merged 5 commits into
mainfrom
ta/SDK-1413/invalid-context-handling-and-logging
Sep 2, 2025
Merged

fix: improves handling of invalid contexts and adds SSE Client logging.#207
tanderson-ld merged 5 commits into
mainfrom
ta/SDK-1413/invalid-context-handling-and-logging

Conversation

@tanderson-ld

@tanderson-ld tanderson-ld commented Aug 29, 2025

Copy link
Copy Markdown
Contributor

Requirements

  • I have added test coverage for new or changed functionality

  • I have followed the repository's pull request submission guidelines

  • I have validated my changes against all supported platform versions

Related issues

SDK-1413

Describe the solution you've provided

Adds context validity checks to start and identify that prevents state change if invalid context is provided.
Adds logging to SSE client and necessary adapter to use LDLogger.

@tanderson-ld
tanderson-ld requested a review from a team as a code owner August 29, 2025 21:01
Comment thread packages/event_source_client/lib/src/sse_client_html.dart Outdated
Comment thread packages/common_client/lib/src/context_modifiers/anonymous_context_modifier.dart Outdated
Comment thread packages/common_client/lib/src/ld_common_client.dart Outdated
Comment thread packages/common_client/lib/src/ld_common_client.dart Outdated
cursor[bot]

This comment was marked as outdated.

Comment thread packages/common_client/lib/src/ld_common_client.dart
@tanderson-ld
tanderson-ld merged commit fcab81f into main Sep 2, 2025
5 checks passed
@tanderson-ld
tanderson-ld deleted the ta/SDK-1413/invalid-context-handling-and-logging branch September 2, 2025 17:59
tanderson-ld pushed a commit that referenced this pull request Sep 3, 2025
🤖 I have created a release *beep* *boop*
---


##
[1.2.1](launchdarkly_event_source_client-v1.2.0...launchdarkly_event_source_client-v1.2.1)
(2025-09-02)


### Bug Fixes

* improves handling of invalid contexts and adds SSE Client logging.
([#207](#207))
([fcab81f](fcab81f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
tanderson-ld pushed a commit that referenced this pull request Sep 3, 2025
🤖 I have created a release *beep* *boop*
---


##
[1.6.1](launchdarkly_dart_common-v1.6.0...launchdarkly_dart_common-v1.6.1)
(2025-09-03)


### Bug Fixes

* improves handling of invalid contexts and adds SSE Client logging.
([#207](#207))
([fcab81f](fcab81f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
tanderson-ld added a commit that referenced this pull request Sep 3, 2025
🤖 I have created a release *beep* *boop*
---


##
[1.6.2](launchdarkly_common_client-v1.6.1...launchdarkly_common_client-v1.6.2)
(2025-09-03)


### Bug Fixes

* improves handling of invalid contexts and adds SSE Client logging.
([#207](#207))
([fcab81f](fcab81f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: Todd Anderson <127344469+tanderson-ld@users.noreply.github.com>
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.

2 participants