Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Newlines Not Normalized on (64kB) Chunk Boundaries #1815

Closed
awharn opened this issue Oct 4, 2023 · 1 comment · Fixed by #1818
Closed

Newlines Not Normalized on (64kB) Chunk Boundaries #1815

awharn opened this issue Oct 4, 2023 · 1 comment · Fixed by #1818
Labels
bug Something isn't working priority-medium Not functioning - next quarter if capacity permits severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex.

Comments

@awharn
Copy link
Member

awharn commented Oct 4, 2023

Describe the bug

Sometimes, newlines are not normalized and both \r and \n are sent to z/OSMF. It appears that if a file is being uploaded, and \r\n are split across buffers, they are not normalized and a blank line appears.

Expected and actual results

N/A

Describe your environment

Zowe CLI 7.18.6
Windows 10

Additional context
@awharn awharn added bug Something isn't working priority-medium Not functioning - next quarter if capacity permits new The issue wasn't triaged yet severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex. labels Oct 4, 2023
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

Thank you for creating a bug report.
We will investigate the bug and evaluate its impact on the product.
If you haven't already, please ensure you have provided steps to reproduce the bug and as much context as possible.

@awharn awharn changed the title Newlines Not Normalized on 64kB chunk boundaries Newlines Not Normalized on (64kB) Chunk Boundaries Oct 4, 2023
@adam-wolfe adam-wolfe removed the new The issue wasn't triaged yet label Oct 4, 2023
This was referenced Oct 5, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working priority-medium Not functioning - next quarter if capacity permits severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Newline Normalization on Upload (V2) zowe/zowe-cli
2 participants
@awharn @adam-wolfe