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

Fixes #2967 #3061

Merged
merged 5 commits into from
Feb 24, 2025
Merged

Fixes #2967 #3061

merged 5 commits into from
Feb 24, 2025

Conversation

Washbear10
Copy link
Contributor

Details:
This fixes issue #2967 . The line contains an "em-dash" character (—) , which apparently break a powershell string when executed as a script. Strangely enough, running this line manually from a command line session works without complaints.
Testing:
Tested on Windows 11 Enterprise

Associated Issues:
#2967

This fixes issue redcanaryco#2967 . The line contains an "em-dash" character (—) , which apparently break a powershell string when executed as a script. Strangely enough, running this line manually from a command line session works without complaints.
@patel-bhavin
Copy link
Collaborator

@Washbear10 - that is a tough find and thank you for fixing this! Can you attach a screenshot of the execution of this atomic?

@Washbear10
Copy link
Contributor Author

@Washbear10 - that is a tough find and thank you for fixing this! Can you attach a screenshot of the execution of this atomic?

@patel-bhavin here is a screenshot of the command line when running test via Invoke-AtomicTest
Screenshot 2025-02-24 111935

@patel-bhavin
Copy link
Collaborator

@Washbear10 - Thank you for the screenshot!

@patel-bhavin patel-bhavin merged commit 787d043 into redcanaryco:master Feb 24, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants