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.

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

Open pywinrm shell sessions with UTF-8 codepage #6038

Merged
merged 2 commits into from
Oct 17, 2024

Conversation

Stealthii
Copy link
Contributor

This PR implements a change to the shell initialisation process with pywinrm, utilizing the 65001 codepage, ensuring raw string responses are UTF-8 encoded.

Potential fix for #6034.

@pull-request-size pull-request-size bot added the size/M PR that changes 30-99 lines. Good size to review. label Oct 18, 2023
Stealthii added a commit to Stealthii/st2 that referenced this pull request Oct 18, 2023
@Stealthii Stealthii force-pushed the bugfix/winrm_runner-unicode branch from 1755d69 to 173ed22 Compare October 18, 2023 19:34
@Stealthii Stealthii force-pushed the bugfix/winrm_runner-unicode branch from 173ed22 to 5cdc0e7 Compare October 19, 2023 11:26
@cognifloyd cognifloyd added this to the 3.9.0 milestone Feb 19, 2024
@cognifloyd cognifloyd enabled auto-merge February 19, 2024 15:25
With Python 3.6 being the minimum supported version, dropping six here
and utilising 3.x type identifiers.
This change opens a shell with the 65001 codepage, ensuring raw string
responses are UTF-8 encoded.

Fixes StackStorm#6034.
auto-merge was automatically disabled October 16, 2024 19:36

Head branch was pushed to by a user without write access

@Stealthii Stealthii force-pushed the bugfix/winrm_runner-unicode branch from 67fd29f to dcf7c30 Compare October 16, 2024 19:36
@Stealthii
Copy link
Contributor Author

@nzlosh @cognifloyd re-based this PR and fixed the changelog conflicts and formatting standard to match recent entries, apologies for the delay!

@nzlosh nzlosh merged commit 58df5c0 into StackStorm:master Oct 17, 2024
29 checks passed
@nzlosh
Copy link
Contributor

nzlosh commented Oct 17, 2024

Thank you for your help.

@Stealthii Stealthii deleted the bugfix/winrm_runner-unicode branch October 17, 2024 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M PR that changes 30-99 lines. Good size to review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants