Skip to content

fix: cherry pick the fix for java21 encoding issue from V3 into V2 - #5034

Merged
Shobhajayanna merged 2 commits into
v2.x.xfrom
reboot/java21-encoding-fix
Oct 20, 2026
Merged

Shobhajayanna merged 2 commits into
v2.x.xfrom
reboot/java21-encoding-fix

Conversation

@Shobhajayanna

@Shobhajayanna Shobhajayanna commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Description

cherry pick below two PRs from V3 related to java21 encoding issue
#4659
#4482

Type of change

  • fix: Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • PR title conforms to commit message guideline ## Commit Message Structure Guideline
  • I have commented my code, particularly in hard-to-understand areas. In JS I did provide JSDoc
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • The java tests in the area I was working on leverage @nested annotations
  • Any dependent changes have been merged and published in downstream modules

For more details about how should the code look like read the Contributing guideline

hrishikesh-nalawade and others added 2 commits October 19, 2026 15:12
Signed-off-by: hrishikesh-nalawade <hrishikesh.nalawade.17@gmail.com>
Signed-off-by: Hrishikesh Nalawade <167364362+hrishikesh-nalawade@users.noreply.github.com>
Co-authored-by: Pablo Carle <pablocarle@users.noreply.github.com>
(cherry picked from commit 0a636a4)
Signed-off-by: Ian Sergeant <ian.sergeant@broadcom.com>
Signed-off-by: iansergeant42 <129340641+iansergeant42@users.noreply.github.com>
Co-authored-by: Ian Sergeant <ian.sergeant@broadcom.com>
(cherry picked from commit b76ff8a)
@sonarqubecloud

Copy link
Copy Markdown

@iansergeant42 iansergeant42 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Along with what I've suggested, the following files are changed in #4659. I don't know for sure that they exist in v2 though:

apiml-utility/src/main/resources/logback-spring.xml
apiml-utility/src/test/resources/logback-test.xml
discoverable-client/src/main/resources/logback.xml
integration-tests/src/test/resources/logback.xml
onboarding-enabler-micronaut-sample-app/src/main/resources/logback.xml
scripts/test/start_test.bats

Comment thread api-catalog-package/src/main/resources/bin/start.sh Outdated
Comment thread api-catalog-package/src/main/resources/bin/start.sh Outdated
Comment thread caching-service-package/src/main/resources/bin/start.sh Outdated
Comment thread caching-service-package/src/main/resources/bin/start.sh Outdated
Comment thread cloud-gateway-package/src/main/resources/bin/start.sh Outdated
Comment thread discovery-package/src/main/resources/bin/start.sh Outdated
Comment thread gateway-package/src/main/resources/bin/start.sh Outdated
Comment thread gateway-package/src/main/resources/bin/start.sh Outdated
Comment thread metrics-service-package/src/main/resources/bin/start.sh Outdated
Comment thread metrics-service-package/src/main/resources/bin/start.sh Outdated
@Shobhajayanna
Shobhajayanna merged commit 214ded4 into v2.x.x Oct 20, 2026
36 checks passed
@Shobhajayanna
Shobhajayanna deleted the reboot/java21-encoding-fix branch October 20, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants