z/OSMF static definition file processing #4113
Conversation
Signed-off-by: Pablo Carle <[email protected]>
|
build 6636 SUCCEEDED. |
zowe-robot
temporarily deployed
to
marist-zzow11
GitHub Actions
Inactive
|
Test workflow 5847 is started. |
bin/libs/component.ts
Outdated
| @@ -418,6 +418,22 @@ export function processComponentApimlStaticDefinitions(componentDir: string): bo | |||
|
|
|||
| const contents = xplatform.loadFileUTF8(path,xplatform.AUTO_DETECT); | |||
| if (contents) { | |||
| const zosmfScheme = std.getenv("ZOSMF_SCHEME"); | |||
| const attls = std.getenv("ZWE_zowe_network_client_tls_attls"); | |||
There was a problem hiding this comment.
This will not work for configs where people have recommended enabling attls through only the server variable.
It would be better to say if the client variable isn't defined, to use the server variable's value.
Something like we do here:
https://github.com/zowe/zlux-server-framework/blob/v3.x/staging/lib/util.js#L510-L520
Signed-off-by: Pablo Carle <[email protected]>
|
build 6659 FAILED. |
1 similar comment
|
build 6659 FAILED. |
PR type
Changes proposed in this PR
Include logic to determine if connections from API ML components to z/OSMF are made using https (default) or HTTP (when outbound AT-TLS rules are in place)
Does this PR introduce a breaking change?