Skip to content

Conversation

@taban03
Copy link
Contributor

@taban03 taban03 commented Oct 10, 2025

Description

  • There is a conflict on how client certificate is used between GW and ZAAS. In general there are endpoint that requires credentials (basic, JWT), requires any trusted certificate, trusted client certificate and Zowe certificate. The behaviour is little bit different for each type of authentication. The specific impact is from the AT-TLS. Documentation suggest to use an outbound rule signed by certificate. It means that services that requires any trusted certificate starts receiving x509 from AT-TLS layer even it is missing in the original request. It is a known issue that there is no way how to programatically control signing by x509. For these case this fix allows to define header Client-cert (it is used to forward client certificate) with empty value. Empty valud means forward no certificate - actually to remove / ignore the certificate. The usage is only in resending of calls between gateway and ZAAS, but it could be use elsewhere.

  • Add spring components and config to support AT-TLS in DC

Linked to # (issue)
Part of the # (epic)

Type of change

Please delete options that are not relevant.

  • fix: Bug fix (non-breaking change which fixes an issue)
  • feat: New feature (non-breaking change which adds functionality)
  • docs: Change in a documentation
  • refactor: Refactor the code
  • chore: Chore, repository cleanup, updates the dependencies.
  • BREAKING CHANGE or !: Breaking change (fix or feature that would cause existing functionality to not work as expected)

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

@EvaJavornicka EvaJavornicka moved this from New to In Progress in API Mediation Layer Backlog Management Oct 15, 2025
Signed-off-by: Pavel Jareš <Pavel.Jares@broadcom.com>
@pull-request-size pull-request-size bot added size/L and removed size/M labels Oct 15, 2025
@github-actions github-actions bot added the Sensitive Sensitive change that requires peer review label Oct 15, 2025
pavel-jares-bcm and others added 5 commits October 16, 2025 16:50
Signed-off-by: Pavel Jareš <Pavel.Jares@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>

# Conflicts:
#	apiml-tomcat-common/src/main/java/org/zowe/apiml/product/web/TomcatKeyringFix.java
@taban03 taban03 force-pushed the reboot/add_attls_to_dc branch from 62a5360 to 5b7d2a2 Compare October 16, 2025 19:18
@taban03 taban03 changed the title chore: add AT-TLS support to DC fix: Requirement of client certificate on ZAAS call when AT-TLS is used & add AT-TLS support to DC Oct 16, 2025
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
@sonarqubecloud
Copy link

@taban03 taban03 merged commit 47525ee into v3.x.x Oct 17, 2025
99 of 102 checks passed
@taban03 taban03 deleted the reboot/add_attls_to_dc branch October 17, 2025 16:08
Goutham024 pushed a commit to Joe-Winchester/api-layer-OIDC-POC that referenced this pull request Oct 29, 2025
…ed & add AT-TLS support to DC (zowe#4347)

Signed-off-by: Pavel Jareš <Pavel.Jares@broadcom.com>
Signed-off-by: Andrea Tabone <andrea.tabone@broadcom.com>
Co-authored-by: Pavel Jareš <Pavel.Jares@broadcom.com>
Signed-off-by: Gowtham Selvaraj <Gowtham.Selvaraj1@ibm.com>
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/L

Projects

Development

Successfully merging this pull request may close these issues.

4 participants