Here's the bug: http://jira.codehaus.org/browse/GEOS-2281
What I did was exactly what I said I would do; roll back the .available() fix (which had been committed on 1.7.x already) and modify the request logging filter to avoid triggering the problem. This bug is pretty minor (it only shows up when POST body logging is enabled, so there's an easy workaround) so waiting for 1.7.1 is fine. -David Justin Deoliveira wrote: > Hi David, > > I guess I misunderstood our last conversation, I did not think any > commits were going to occur on 1.7.x... and just for 1.7.1. > Regardless, is there a jira issue describing this issue? I don't see > one scheduled against 1.7.0. Or a relevant mailing list thread? > > Sorry to be a nag, I just want to know why we are breaking the freeze > and don't remember any conversation about actual bug. > > -Justin > > [EMAIL PROTECTED] wrote: >> Revision >> 10417 <http://fisheye.codehaus.org/changelog/geoserver/?cs=10417> >> Author >> dwinslow >> Date >> 2008-10-16 20:13:18 -0500 (Thu, 16 Oct 2008) >> >> >> Log Message >> >> The REAL fix for the request logging filter. >> >> >> Modified Paths >> >> * >> branches/1.7.x/geoserver/web/src/main/java/org/geoserver/filters/BufferedRequestStream.java >> >> >> >> <#branches17xgeoserverwebsrcmainjavaorggeoserverfiltersBufferedRequestStreamjava> >> >> >> >> >> Diff >> >> >> Modified: >> >> branches/1.7.x/geoserver/web/src/main/java/org/geoserver/filters/BufferedRequestStream.java >> >> >> (10416 => 10417) >> >> --- >> branches/1.7.x/geoserver/web/src/main/java/org/geoserver/filters/BufferedRequestStream.java >> >> 2008-10-16 23:59:44 UTC (rev 10416) >> +++ >> branches/1.7.x/geoserver/web/src/main/java/org/geoserver/filters/BufferedRequestStream.java >> >> 2008-10-17 01:13:18 UTC (rev 10417) >> @@ -1,23 +1,22 @@ >> package org.geoserver.filters; >> >> import javax.servlet.ServletInputStream; >> +import java.io.ByteArrayInputStream; >> import java.io.InputStream; >> import java.io.IOException; >> -import java.io.StringReader; >> -import java.io.Reader; >> >> /** >> * Wrap a String up as a ServletInputStream so we can read it >> multiple times. >> * @author David Winslow <[EMAIL PROTECTED]> >> */ >> public class BufferedRequestStream extends ServletInputStream{ >> - Reader myReader; >> + InputStream myStream; >> >> public BufferedRequestStream(String buff) throws IOException { >> - myReader = new StringReader(buff); >> - myReader.mark(16); >> - myReader.read(); >> - myReader.reset(); >> + myStream = new ByteArrayInputStream(buff.getBytes()); >> + myStream.mark(16); >> + myStream.read(); >> + myStream.reset(); >> } >> >> public int readLine(byte[] b, int off, int len) throws IOException{ >> @@ -26,7 +25,7 @@ >> int end = off + len; >> >> while (index < end && - (read = >> myReader.read()) != -1){ >> + (read = myStream.read()) != -1){ >> b[index] = (byte)read; index++; >> if (((char)read)== '\n'){ >> @@ -38,6 +37,10 @@ >> } >> >> public int read() throws IOException{ >> - return myReader.read(); >> + return myStream.read(); >> } >> + >> + public int available() throws IOException { >> + return myStream.available(); >> + } >> } >> >> ------------------------------------------------------------------------ >> >> To unsubscribe from this list please visit: >> >> http://xircles.codehaus.org/manage_email >> > > ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Geoserver-devel mailing list Geoserver-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geoserver-devel
