possible bug in HttpRecordingImplementation.java line 538?

if (!request.getUri().getExtends().equals(lastBaseURI) ||
                request.isSetBody() ||
==>        !(m_resourcePathPattern.matcher(request.getUri().getUnparsed()).matches() ||
                 lastResponseWasRedirect) ||
                currentPage == null) {
  currentPage = httpRecording.addNewPage();
}


Notice the ! () around both the resource path test and redirect test:
!(m_resourcePathPattern.matcher(request.getUri().getUnparsed()).matches() || lastResponseWasRedirect)

Perhaps this should look like this?

if (!request.getUri().getExtends().equals(lastBaseURI) ||
                request.isSetBody() ||
                !m_resourcePathPattern.matcher(request.getUri().getUnparsed()).matches() ||
                lastResponseWasRedirect ||
                currentPage == null) {
  currentPage = httpRecording.addNewPage();
}


James

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1

_______________________________________________
Grinder-development mailing list
Grinder-development-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/grinder-development