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

add logging in pythonw env and fix connection drops with it #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

razuz
Copy link

@razuz razuz commented Aug 8, 2016

Long debugged error which caused much misery :)

Anyways stderr (and also stdout) needs some sort of env which is set to None in pythonw.exe ... this however causes weird anomalies while talking to agent :/

This is one solution to fix this issue - there are probably more ways to handle that but as far as I tested then this works.

@jbremer
Copy link
Owner

jbremer commented Aug 8, 2016

Furthermore, this doesn't work if people rename pythonw.exe. What do you think about doing this at all times - no matter python.exe or pythonw.exe?

@razuz
Copy link
Author

razuz commented Aug 8, 2016

sure why not ... the only downside would be that log will be written to file not console

I'll test this later today on vmcloak and cuckoo - just to be safe

@razuz
Copy link
Author

razuz commented Aug 8, 2016

this seems to work both ways ... haven't tested with cuckoo yet - will do that in morning

@razuz
Copy link
Author

razuz commented Aug 9, 2016

works in cuckoo aswell :)
I don't see any more connection drops on cuckoo which probably solves the file download anomaly we were looking at earlier ;)

so I guess it's good for merge

@jbremer jbremer force-pushed the master branch 14 times, most recently from 0a21d8a to 8799c64 Compare October 23, 2016 19:37
federicofantini referenced this pull request in certego/agent Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants