fix: Bump java-sdk-internal to 1.12.2 for redactedAttributes escaping - #203
Merged
Merged
Conversation
…alpha Co-Authored-By: jbailey@launchdarkly.com <jbailey@launchdarkly.com>
Contributor
Author
🤖 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:
|
Contributor
Author
|
@cursor review |
Co-Authored-By: jbailey@launchdarkly.com <jbailey@launchdarkly.com>
The server SDK consumes java-sdk-internal as a published artifact, so the redactedAttributes escaping fix only reaches it through a version bump. This lets the v3 contract tests covering escaped attribute references pass.
jsonbailey
marked this pull request as ready for review
September 30, 2026 18:31
jsonbailey
approved these changes
Sep 30, 2026
jsonbailey
changed the title
tanderson-ld
approved these changes
Sep 30, 2026
Contributor
Author
|
❌ Cannot revive Devin session - the session is too old. Please start a new session instead. |
jsonbailey
pushed a commit
that referenced
this pull request
Sep 30, 2026
🤖 I have created a release *beep* *boop* --- ## [7.17.2](launchdarkly-java-server-sdk-7.17.1...launchdarkly-java-server-sdk-7.17.2) (2026-09-30) ### Bug Fixes * Bump java-sdk-internal to 1.12.2 for redactedAttributes escaping ([#203](#203)) ([4c0c7bd](4c0c7bd)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Overview** > **Patch release** that bumps **launchdarkly-java-server-sdk** from **7.17.1** to **7.17.2** via Release Please (manifest, `gradle.properties`, `Version.SDK_VERSION`, and changelog). > > The only functional note in this release is pulling in **`launchdarkly-java-sdk-internal` 1.12.2**, which fixes **escaping of attribute names** listed in event context `_meta.redactedAttributes` ([#203](#203)). > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit 31e6fee. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
chore: Track the latest v3 contract test release instead of a pinned alpha
Bumps
java-sdk-internalto 1.12.2 so the server SDK picks up theredactedAttributesescaping fix from #228. The server SDK consumes internal as a published artifact, so the fix only reaches it through this bump — that is what clears the 14 failing v3 contract tests.Also moves the v3 contract tests off the pinned
v3.0.0-alpha.6onto floatingv3, so new harness releases are picked up without a manual bump each time. The FDv1 job has always floated onv2.SDK-2922
Link to Devin session: https://app.devin.ai/sessions/6409748db442482fa8ddc43061f0277d
Requested by: @jsonbailey