Router should not redirect System.out and System.err

[ocean1]

PROBLEM:

Right now when a Router starts up it will redirect System.out and System.err to the wrapper.log file. This plays havoc with an application that embeds the Router.

See this code in WorkingDir?.java: ​https://github.com/i2p/i2p.i2p/blob/master/router/java/src/net/i2p/router/startup/WorkingDir.java#L277

WORK-AROUND:

Applications can short-circuit this behavior using the code:

System.setProperty("wrapper.version", "NO_WRAPPER");

PROPOSED SOLUTION:

Ideally the router would look for positive confirmation that a wrapper is doing is being used before redirecting System.out and System.err. There should perhaps be a system property and/or a static method that tests whether a wrapper is in use.

[zzz]

We do look for "positive confirmation" - that's the check for wrapper.version. That IS the system property.

However, there are other places in the code that rely on that also, so the workaround isn't great.

We'd be happy to add a new property, similar to the other ones (see Router.java line 130 if you haven't yet). Probably I2P_DISABLE_OUTPUT_OVERRIDE

[zzz]

In 2a6889552fa84500fb788acca7263ddecc96d791 0.9.31-2