Skip to content

Conversation

pj892031
Copy link
Contributor

@pj892031 pj892031 commented Oct 1, 2025

Description

The original issue was when a customer configure zowe.yml to use keyring and provide the keyring URL with just douple slashes (not four). All services except the caching service fix the issue in Tomcat. The handler responsible to fixing URLs was not applied for the caching service, because the implementation is only for servlet implementation. Caching service and API catalog uses WebFlux. Therefor the handler was ignored and not applied. The workarround was to use 4 slashes. More information about the error itself is in the linked issue.

Linked to #4324

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

Signed-off-by: Pavel Jareš <[email protected]>
@pablocarle
Copy link
Contributor

Can you decribe the issue in the PR description?

@pj892031
Copy link
Contributor Author

pj892031 commented Oct 8, 2025

Can you decribe the issue in the PR description?

I have added short description.

@EvaJavornicka EvaJavornicka moved this from New to In Progress in API Mediation Layer Backlog Management Oct 8, 2025
@pull-request-size pull-request-size bot added size/L and removed size/S labels Oct 8, 2025
@pull-request-size pull-request-size bot added size/XL and removed size/L labels Oct 9, 2025
Copy link
Member

@achmelo achmelo left a comment

Choose a reason for hiding this comment

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

Please don't forget to set infinispan log levels

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
50.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@pj892031 pj892031 merged commit 3bff245 into v3.x.x Oct 10, 2025
37 of 38 checks passed
@pj892031 pj892031 deleted the reboot/tomcat-customizer-for-reactive branch October 10, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Sensitive Sensitive change that requires peer review size/XL

Projects

Development

Successfully merging this pull request may close these issues.

3 participants