Store logs to a file

[pditommaso]

Being an application meant to be executed in the background, it would make sense to redirect application logs to a file .tw-agent.log and render to console only logs with level >= INFO.

[pditommaso]

Related to this, it would be better to not log execution commands that can contain encoded variables, for example:

15:52:20.794 [default-nioEventLoopGroup-1-3] INFO  io.seqera.tower.agent.Agent - Execute: mkdir -p /home/ubuntu/work; 
cd /home/ubuntu/work;
echo IyEvYmluL2Jhc2gKI1NCQVRDSCAtRCAvaG9tZS91YnVudHUvd29yawojU0JBVENIIC1KIG5mLXdvcmtmbG93LVFHTmFiSlB5UFg0QVEKI1NCQVRDSCAtbyAvaG9tZS91YnVudHUvd29yay9uZi1RR05hYkpQeVBYNEFRLmxvZwojU0JBVENIIC0tbm8tcmVxdWV1ZQojU0JBVENIIC1wIGxvbmcKc2V0IC1lCnNldCAtbyBwaXBlZmFpbAoKIyBJbnB1dCB2YXJpYWJsZXM6CiMKIyAtIE5YRl9VVUlEOiBuZXh0ZmxvdyBzZXNzaW9uIGlkIGdlbmVyYXRlZCBieSB0b3dlcgojIC0gTlh.. etc

Ideally this should only be logged when TRACE level is enabled not default (debug) logging

[jordeu]

To log to a file causes this error when building the GraalVM static image: micronaut-projects/micronaut-core#3683

It seems that there is a workaround that may be valid for our use case. Even though, I'm moving this issue to the next Agent release v0.5 because this requires further testing.

The part of removing sensitive data it is implemented on v0.4

[pditommaso]

Try with log4j 😄

[pditommaso]

however the most important thing here is to make it less verbose. can you at least not dump executed command and result to console, maybe just changing log.debug to log.trace level?

[jordeu]

however the most important thing here is to make it less verbose. can you at least not dump executed command and result to console, maybe just changing log.debug to log.trace level?

I'm doing this part. See #32