diff --git a/scm-core/src/main/java/sonia/scm/logging/LoggingOutputStream.java b/scm-core/src/main/java/sonia/scm/logging/LoggingOutputStream.java
new file mode 100644
index 0000000000..bcf232400a
--- /dev/null
+++ b/scm-core/src/main/java/sonia/scm/logging/LoggingOutputStream.java
@@ -0,0 +1,275 @@
+/**
+ * Copyright (c) 2010, Sebastian Sdorra
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *
+ * 1. Redistributions of source code must retain the above copyright notice,
+ * this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright notice,
+ * this list of conditions and the following disclaimer in the documentation
+ * and/or other materials provided with the distribution.
+ * 3. Neither the name of SCM-Manager; nor the names of its
+ * contributors may be used to endorse or promote products derived from this
+ * software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE FOR ANY
+ * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON
+ * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * http://bitbucket.org/sdorra/scm-manager
+ *
+ */
+
+
+
+package sonia.scm.logging;
+
+//~--- non-JDK imports --------------------------------------------------------
+
+import org.slf4j.Logger;
+
+//~--- JDK imports ------------------------------------------------------------
+
+import java.io.IOException;
+import java.io.OutputStream;
+
+/**
+ * Based on LoggingOutputStream by Jim Moore
+ * @author Sebastian Sdorra
+ */
+public class LoggingOutputStream extends OutputStream
+{
+
+ /**
+ * The default number of bytes in the buffer.
+ */
+ public static final int DEFAULT_BUFFER_LENGTH = 2048;
+
+ /** Field description */
+ public static final int LEVEL_DEBUG = 1;
+
+ /** Field description */
+ public static final int LEVEL_ERROR = 4;
+
+ /** Field description */
+ public static final int LEVEL_INFO = 2;
+
+ /** Field description */
+ public static final int LEVEL_TRACE = 0;
+
+ /** Field description */
+ public static final int LEVEL_WARN = 3;
+
+ //~--- constructors ---------------------------------------------------------
+
+ /**
+ * Creates the LoggingOutputStream to flush to the given Category.
+ *
+ * @param logger the Logger to write to
+ * @param level the Level to use when writing to the Logger
+ * @throws IllegalArgumentException if cat == null or priority ==
+ * null
+ */
+ public LoggingOutputStream(Logger logger, int level)
+ throws IllegalArgumentException
+ {
+ if (logger == null)
+ {
+ throw new IllegalArgumentException("cat == null");
+ }
+
+ this.level = level;
+ this.logger = logger;
+ bufferLength = DEFAULT_BUFFER_LENGTH;
+ buffer = new byte[DEFAULT_BUFFER_LENGTH];
+ count = 0;
+ }
+
+ //~--- methods --------------------------------------------------------------
+
+ /**
+ * Closes this output stream and releases any system resources
+ * associated with this stream. The general contract of
+ * close
+ * is that it closes the output stream. A closed stream cannot
+ * perform
+ * output operations and cannot be reopened.
+ */
+ @Override
+ public void close()
+ {
+ flush();
+ closed = true;
+ }
+
+ /**
+ * Flushes this output stream and forces any buffered output bytes
+ * to be written out. The general contract of flush is
+ * that calling it is an indication that, if any bytes previously
+ * written have been buffered by the implementation of the output
+ * stream, such bytes should immediately be written to their
+ * intended destination.
+ */
+ @Override
+ public void flush()
+ {
+ if (count == 0)
+ {
+ return;
+ }
+
+ // don't print out blank lines; flushing from PrintStream puts
+ // For linux system
+ if ((count == 1) && ((char) buffer[0]) == '\n')
+ {
+ reset();
+
+ return;
+ }
+
+ // For mac system
+ if ((count == 1) && ((char) buffer[0]) == '\r')
+ {
+ reset();
+
+ return;
+ }
+
+ // On windows system
+ if ((count == 2) && (char) buffer[0] == '\r' && (char) buffer[1] == '\n')
+ {
+ reset();
+
+ return;
+ }
+
+ final byte[] theBytes = new byte[count];
+
+ System.arraycopy(buffer, 0, theBytes, 0, count);
+ log(new String(theBytes));
+ reset();
+ }
+
+ /**
+ * Method description
+ *
+ *
+ * @param message
+ */
+ public void log(String message)
+ {
+ switch (level)
+ {
+ case LEVEL_TRACE :
+ logger.trace(message);
+
+ break;
+
+ case LEVEL_DEBUG :
+ logger.debug(message);
+
+ break;
+
+ case LEVEL_INFO :
+ logger.info(message);
+
+ break;
+
+ case LEVEL_WARN :
+ logger.warn(message);
+
+ break;
+
+ case LEVEL_ERROR :
+ logger.error(message);
+
+ break;
+
+ default :
+ logger.warn(message);
+ }
+ }
+
+ /**
+ * Writes the specified byte to this output stream. The general
+ * contract for write is that one byte is written
+ * to the output stream. The byte to be written is the eight
+ * low-order bits of the argument b. The 24
+ * high-order bits of b are ignored.
+ *
+ * @param b the byte to write
+ *
+ * @throws IOException
+ */
+ @Override
+ public void write(final int b) throws IOException
+ {
+ if (closed)
+ {
+ throw new IOException("The stream has been closed.");
+ }
+
+ // would this be writing past the buffer?
+ if (count == bufferLength)
+ {
+
+ // grow the buffer
+ final int newBufLength = bufferLength + DEFAULT_BUFFER_LENGTH;
+ final byte[] newBuf = new byte[newBufLength];
+
+ System.arraycopy(buffer, 0, newBuf, 0, bufferLength);
+ buffer = newBuf;
+ bufferLength = newBufLength;
+ }
+
+ buffer[count] = (byte) b;
+ count++;
+ }
+
+ /**
+ * Method description
+ *
+ */
+ private void reset()
+ {
+ count = 0;
+ }
+
+ //~--- fields ---------------------------------------------------------------
+
+ /**
+ * The internal buffer where data is stored.
+ */
+ private byte[] buffer;
+
+ /**
+ * Remembers the size of the buffer for speed.
+ */
+ private int bufferLength;
+
+ /** Field description */
+ private boolean closed = false;
+
+ /**
+ * The number of valid bytes in the buffer. This value is always
+ * in the range 0 through buf.length; elements
+ * buf[0] through buf[count-1] contain valid
+ * byte data.
+ */
+ private int count;
+
+ /** Field description */
+ private int level = -1;
+
+ /** The logger to write to. */
+ private Logger logger;
+}
diff --git a/scm-core/src/main/java/sonia/scm/util/IOUtil.java b/scm-core/src/main/java/sonia/scm/util/IOUtil.java
index cdc9350ee8..210028b905 100644
--- a/scm-core/src/main/java/sonia/scm/util/IOUtil.java
+++ b/scm-core/src/main/java/sonia/scm/util/IOUtil.java
@@ -124,6 +124,50 @@ public class IOUtil
}
}
+ /**
+ * Method description
+ *
+ *
+ * @param source
+ * @param target
+ *
+ * @throws IOException
+ */
+ public static void copy(File source, File target) throws IOException
+ {
+ if (source.isDirectory())
+ {
+ if (!target.exists())
+ {
+ mkdirs(target);
+ }
+
+ String[] children = source.list();
+
+ for (String child : children)
+ {
+ copy(new File(source, child), new File(target, child));
+ }
+ }
+ else
+ {
+ FileInputStream input = null;
+ FileOutputStream output = null;
+
+ try
+ {
+ input = new FileInputStream(source);
+ output = new FileOutputStream(target);
+ copy(input, output);
+ }
+ finally
+ {
+ close(input);
+ close(output);
+ }
+ }
+ }
+
/**
* Method description
*
@@ -136,6 +180,18 @@ public class IOUtil
new Thread(new IOCopyThread(reader, writer)).start();
}
+ /**
+ * Method description
+ *
+ *
+ * @param input
+ * @param output
+ */
+ public static void copyThread(InputStream input, OutputStream output)
+ {
+ new Thread(new IOStreamCopyThread(input, output)).start();
+ }
+
/**
* Method description
*
@@ -165,39 +221,6 @@ public class IOUtil
}
}
- public static void copy( File source, File target ) throws IOException
- {
- if ( source.isDirectory() )
- {
- if ( ! target.exists() )
- {
- mkdirs(target);
- }
-
- String[] children = source.list();
- for ( String child : children )
- {
- copy(new File(source, child), new File(target, child));
- }
- }
- else
- {
- FileInputStream input = null;
- FileOutputStream output = null;
- try
- {
- input = new FileInputStream( source );
- output = new FileOutputStream( target );
- copy(input, output);
- }
- finally
- {
- close(input);
- close(output);
- }
- }
- }
-
/**
* Method description
*
@@ -294,4 +317,61 @@ public class IOUtil
/** Field description */
private Writer writer;
}
+
+
+ /**
+ * Class description
+ *
+ *
+ * @version Enter version here..., 2010-12-27
+ * @author Sebastian Sdorra
+ */
+ private static class IOStreamCopyThread implements Runnable
+ {
+
+ /**
+ * Constructs ...
+ *
+ *
+ * @param input
+ * @param output
+ */
+ public IOStreamCopyThread(InputStream input, OutputStream output)
+ {
+ this.input = input;
+ this.output = output;
+ }
+
+ //~--- methods ------------------------------------------------------------
+
+ /**
+ * Method description
+ *
+ */
+ @Override
+ public void run()
+ {
+ try
+ {
+ copy(input, output);
+ }
+ catch (IOException ex)
+ {
+ logger.error(ex.getMessage(), ex);
+ }
+ finally
+ {
+ close(input);
+ close(output);
+ }
+ }
+
+ //~--- fields -------------------------------------------------------------
+
+ /** Field description */
+ private InputStream input;
+
+ /** Field description */
+ private OutputStream output;
+ }
}
diff --git a/scm-core/src/main/java/sonia/scm/web/cgi/CGIRunner.java b/scm-core/src/main/java/sonia/scm/web/cgi/CGIRunner.java
index 7b2d197b72..9aabdf7a85 100644
--- a/scm-core/src/main/java/sonia/scm/web/cgi/CGIRunner.java
+++ b/scm-core/src/main/java/sonia/scm/web/cgi/CGIRunner.java
@@ -38,6 +38,7 @@ package sonia.scm.web.cgi;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+import sonia.scm.logging.LoggingOutputStream;
import sonia.scm.util.IOUtil;
import sonia.scm.util.Util;
@@ -223,8 +224,9 @@ public class CGIRunner
final int inLength = len;
// TODO: print or log error stream
- IOUtil.copyThread(new InputStreamReader(p.getErrorStream()),
- new OutputStreamWriter(System.err));
+ IOUtil.copyThread(p.getErrorStream(),
+ new LoggingOutputStream(logger,
+ LoggingOutputStream.LEVEL_ERROR));
new Thread(new Runnable()
{
@Override