zowe / zowe-common-c

C Libraries for various OS & Networking needs
Eclipse Public License 2.0
10 stars 30 forks source link

httpClientSessionInit does not return rc/rsn to callers #443

Open 1000TurquoisePogs opened 5 months ago

1000TurquoisePogs commented 5 months ago

httpClientSessionInit has its own set of status values it returns, but the actual rc/rsn of commands it does within are hidden behind compile-time printf statements.

This is unhelpful because this function does fail sometimes, and on zOS the reason is opaque and requires a GSK trace to then discover the cause. It just costs extra debug time then.

Is there any opposition to just add more arguments to httpClientSessionInit to allow passing back rc/rsn?

https://github.com/zowe/zowe-common-c/blob/v2.x/staging/c/httpclient.c#L652

JoeNemo commented 4 months ago

If @DivergentEuropeans or @Gautham-coder want this one to fix, please take it. It's probably not hard.

Gautham-coder commented 4 months ago

I will take it up Joe.