Skip to content

Encoding header for dsn get set incorrectly#453

Merged
frankgiordano merged 1 commit intozowe:mainfrom
KasperHelsted:bugfix/invalidDnsGetEncodeingHeader
Jan 22, 2026
Merged

Encoding header for dsn get set incorrectly#453
frankgiordano merged 1 commit intozowe:mainfrom
KasperHelsted:bugfix/invalidDnsGetEncodeingHeader

Conversation

@KasperHelsted
Copy link
Copy Markdown
Contributor

While fetching data using the Zowe Java SDK, I noticed that Danish characters were rendered incorrectly.

To verify whether this was client-specific, I performed the same operation using Zowe Explorer in Visual Studio Code, where the characters appeared correctly.

I then inspected the HTTP requests sent by both clients and observed a difference in the headers. Requests from Visual Studio Code included:
X-IBM-Data-Type: text;fileEncoding=IBM-277

In contrast, requests made via the Java SDK contained:
X_IBM_TEXT: X_IBM_TEXTX_IBM_TEXT_ENCODINGOptionalLong[277]

This header format did not appear to be correct.

This PR updates the header generation to align with the structure used in the other DsnGet cases. After applying this change, I verified that Danish characters are now rendered correctly when calling Zowe through the Java SDK.

…test

Signed-off-by: Kasper Helsted <helsted@vopu.dk>
@KasperHelsted KasperHelsted force-pushed the bugfix/invalidDnsGetEncodeingHeader branch from 730564d to ab20c6d Compare January 21, 2026 10:40
@frankgiordano frankgiordano self-requested a review January 22, 2026 02:30
@frankgiordano frankgiordano added the bug Something isn't working label Jan 22, 2026
@frankgiordano
Copy link
Copy Markdown
Collaborator

Thank you for this contribution! I will release this fix in an upcoming release 6.0.4.

@frankgiordano frankgiordano merged commit b4dbd22 into zowe:main Jan 22, 2026
2 checks passed
@KasperHelsted KasperHelsted deleted the bugfix/invalidDnsGetEncodeingHeader branch January 22, 2026 06:13
@frankgiordano
Copy link
Copy Markdown
Collaborator

@KasperHelsted This is now included in the new 6.1.0 just published. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants