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 an option for AttachMe to be quiet on stdout/stderr #8

Open
smola opened this issue Oct 22, 2021 · 5 comments
Open

Add an option for AttachMe to be quiet on stdout/stderr #8

smola opened this issue Oct 22, 2021 · 5 comments

Comments

@smola
Copy link

smola commented Oct 22, 2021

I'm debugging JVM processes that get called by other programs and whose stdout/stderr get examined. AttachMe messages to stdout/stderr break the calling process by introducing unexpected output.

It would be nice to add an additional agent option (maybe quiet=[y/n]) to avoid printing anything, at least for anything that is not a fatal error. That would make it possible to debug JVM cli apps that get called from scripts, etc.

@samvel1024
Copy link
Contributor

samvel1024 commented Oct 26, 2021

@smola I think attachme agent should be writing only to stderr. Is it still an issue for you?

Let me know in case stdout gets polluted with attachme messages as well.

@smola
Copy link
Author

smola commented Jan 10, 2022

Unfortunately, stderr pollution is also a problem in my use case, since the caller is a 3rd party which captures both stdout and stderr.

@big-andy-coates
Copy link

@smola what's the 'unexpected output' you're seeing? Is it "Listening for transport dt_socket at address: "? If it isn, then this is related to #11, which has a workaround noted and a PR coming.

@smola
Copy link
Author

smola commented Dec 23, 2022

@big-andy-coates Yes, that was the problem. I was using this in a subprocess, and the parent process did not expect the child to have any extra output.

@big-andy-coates
Copy link

Great.

Maybe close this issue as a duplicate then?

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

No branches or pull requests

3 participants