Skip to content

Conversation

@pablocarle
Copy link
Contributor

@pablocarle pablocarle commented Sep 24, 2025

Description

z/OSMF can be configured to use https or http scheme even if AT-TLS is enabled at the Zowe level.
This PR fixes some missing properties to properly handle these scenarios.

Related to zowe/zowe-install-packaging#4512

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: Pablo Carle <pablo.carle@broadcom.com>
@sonarqubecloud
Copy link

@nxhafa nxhafa merged commit ca8a9ad into v3.x.x Sep 26, 2025
69 of 70 checks passed
@nxhafa nxhafa deleted the reboot/fix/attls-zosmf branch September 26, 2025 10:26
nxhafa added a commit that referenced this pull request Sep 26, 2025
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Co-authored-by: Pablo Carle <pablo.carle@broadcom.com>
Co-authored-by: Nafi Xhafa <164854562+nxhafa@users.noreply.github.com>
Goutham024 pushed a commit to Joe-Winchester/api-layer-OIDC-POC that referenced this pull request Oct 29, 2025
Signed-off-by: Pablo Carle <pablo.carle@broadcom.com>
Co-authored-by: Pablo Carle <pablo.carle@broadcom.com>
Co-authored-by: Nafi Xhafa <164854562+nxhafa@users.noreply.github.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

Projects

Development

Successfully merging this pull request may close these issues.

4 participants