[
https://issues.apache.org/jira/browse/LOG4J2-514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14260529#comment-14260529
]
Gary Gregory commented on LOG4J2-514:
-------------------------------------
[~bruce.brouwer], [~wolf480pl]:
With the test only patch LOG4J2-514-gg-test-only-2014-12-29.diff, the new test
passes. So the test needs updating.
Can you or Bruce take the time to update the test so that it fails with the
Bruce's fix? It would be good if you could also adjust Bruce's code to the
current code in master.
> ConsoleAppender closing System.out on Windows
> ---------------------------------------------
>
> Key: LOG4J2-514
> URL: https://issues.apache.org/jira/browse/LOG4J2-514
> Project: Log4j 2
> Issue Type: Bug
> Components: Appenders
> Affects Versions: 2.0-beta9, 2.0.2, 2.1
> Environment: Windows 7 64-bit
> Oracle JDK 1.7.0_51
> Reporter: Wolf480 Pl
> Assignee: Bruce Brouwer
> Attachments: LOG4J2-514.patch.txt
>
>
> If the Console appender is not specified in the configuration file, after the
> configuration is read, the Console appender is being destroyed, which causes
> it to release its OutputStreamManager, When this manager is released and
> closed, it checks if the stream is System.out or System.err, and if it is,
> doesn't close it. But on Windows, ConsoleAppender wraps the System.out into a
> WindowsAnsiOutputStream. The OutputStreamManager closes the stream when it's
> released, and the stream closes the underlaying System.out.
> proof: http://imageshack.com/a/img31/8296/cg6b.png
> -I think the easiest solution would be to use new
> FileOutputStream(FileDescriptor.out)) instead of System.out in
> ConsoleAppender implementation.- unfortunately, that would still cause
> System.out to be closed.
> A solution would be to wrap System.out with some FilterStream that overrides
> close() to do nothing.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
