fixes from merge
This commit is contained in:
parent
758ce66c8d
commit
7e68a98dcd
|
@ -74,7 +74,7 @@ public class BadRequestLogHandlerTest
|
||||||
@Override
|
@Override
|
||||||
public void log(Request request, Response response)
|
public void log(Request request, Response response)
|
||||||
{
|
{
|
||||||
captured.add(String.format("%s %s %s %03d",request.getMethod(),request.getUri().toString(),request.getProtocol(),response.getStatus()));
|
captured.add(String.format("%s %s %s %03d",request.getMethod(),request.getHttpURI(),request.getProtocol(),response.getStatus()));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -67,7 +67,7 @@ public class RequestLogHandlerTest
|
||||||
@Override
|
@Override
|
||||||
public void log(Request request, Response response)
|
public void log(Request request, Response response)
|
||||||
{
|
{
|
||||||
captured.add(String.format("%s %s %s %03d",request.getMethod(),request.getUri().toString(),request.getProtocol(),response.getStatus()));
|
captured.add(String.format("%s %s %s %03d",request.getMethod(),request.getHttpURI().toString(),request.getProtocol(),response.getStatus()));
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -33,6 +33,7 @@ import javax.servlet.http.HttpServletResponse;
|
||||||
import org.eclipse.jetty.client.HttpClient;
|
import org.eclipse.jetty.client.HttpClient;
|
||||||
import org.eclipse.jetty.client.api.ContentResponse;
|
import org.eclipse.jetty.client.api.ContentResponse;
|
||||||
import org.eclipse.jetty.client.api.Request;
|
import org.eclipse.jetty.client.api.Request;
|
||||||
|
import org.eclipse.jetty.http.HttpHeader;
|
||||||
import org.eclipse.jetty.nosql.NoSqlSession;
|
import org.eclipse.jetty.nosql.NoSqlSession;
|
||||||
import org.eclipse.jetty.server.session.AbstractTestServer;
|
import org.eclipse.jetty.server.session.AbstractTestServer;
|
||||||
import org.junit.Test;
|
import org.junit.Test;
|
||||||
|
@ -94,7 +95,8 @@ public class AttributeNameTest
|
||||||
assertEquals("a.b.c",sessionTestResponse[0]);
|
assertEquals("a.b.c",sessionTestResponse[0]);
|
||||||
|
|
||||||
|
|
||||||
String sessionCookie = response.getHeaders().getStringField("Set-Cookie");
|
String sessionCookie = response.getHeaders().get(HttpHeader.SET_COOKIE);
|
||||||
|
|
||||||
assertTrue(sessionCookie != null);
|
assertTrue(sessionCookie != null);
|
||||||
//Mangle the cookie, replacing Path with $Path, etc.
|
//Mangle the cookie, replacing Path with $Path, etc.
|
||||||
sessionCookie = sessionCookie.replaceFirst("(\\W)(P|p)ath=","$1\\$Path=");
|
sessionCookie = sessionCookie.replaceFirst("(\\W)(P|p)ath=","$1\\$Path=");
|
||||||
|
|
Loading…
Reference in New Issue