Skip to content

Conversation

NeatGuyCoding
Copy link
Contributor

@NeatGuyCoding NeatGuyCoding commented Oct 5, 2025

Make sure that:

  • You have read the contribution guidelines.
  • You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
  • You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.

In src/main/java/io/lettuce/core/json/DelegateJsonArray.java around lines 61-63, asList() currently constructs elements using new DelegateJsonValue(jsonNode,objectMapper) which breaks consistency with get(int) that returns wrapped DelegateJsonObject/DelegateJsonArray for nested structures

Add unit tests for those scenarios

    DelegateJsonObject/DelegateJsonArray for nested structures

Signed-off-by: NeatGuyCoding <15627489+NeatGuyCoding@users.noreply.github.com>
Copy link

jit-ci bot commented Oct 5, 2025

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

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.

1 participant