archive-access-cvs

[Archive-access-cvs] SF.net SVN: archive-access:[3160] trunk/archive-access/projects/wayback/ wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ ArchivalUrlFormRequestParser.java

[<bra...@us...>]

Revision: 3160
          http://archive-access.svn.sourceforge.net/archive-access/?rev=3160&view=rev
Author:   bradtofel
Date:     2010-06-24 20:24:09 +0000 (Thu, 24 Jun 2010)

Log Message:
-----------
BUGFIX(unreported) bad "if" logic...

Modified Paths:
--------------
    trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ArchivalUrlFormRequestParser.java

Modified: trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ArchivalUrlFormRequestParser.java
===================================================================
--- trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ArchivalUrlFormRequestParser.java	2010-06-24 20:23:06 UTC (rev 3159)
+++ trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ArchivalUrlFormRequestParser.java	2010-06-24 20:24:09 UTC (rev 3160)
@@ -50,7 +50,7 @@
 		WaybackRequest wbRequest = super.parse(httpRequest, accessPoint);
 		if(wbRequest != null) {
 			String replayTimestamp = wbRequest.getReplayTimestamp();
-			if((replayTimestamp != null) && replayTimestamp.length() == 0) {
+			if((replayTimestamp == null) || replayTimestamp.length() == 0) {
 				// lets call it a star query:
 				// TODO: should we clone?
 				wbRequest.setStartTimestamp(null);


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

[Archive-access-cvs] SF.net SVN: archive-access:[3533] trunk/archive-access/projects/wayback/ wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ ArchivalUrlFormRequestParser.java

[<bra...@us...>]

Revision: 3533
          http://archive-access.svn.sourceforge.net/archive-access/?rev=3533&view=rev
Author:   bradtofel
Date:     2011-09-06 04:20:57 +0000 (Tue, 06 Sep 2011)
Log Message:
-----------
FEATURE: checks with AccessPoint before forcing "clean queries"

Modified Paths:
--------------
    trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ArchivalUrlFormRequestParser.java

Modified: trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ArchivalUrlFormRequestParser.java
===================================================================
--- trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ArchivalUrlFormRequestParser.java	2011-09-06 04:20:15 UTC (rev 3532)
+++ trunk/archive-access/projects/wayback/wayback-core/src/main/java/org/archive/wayback/archivalurl/requestparser/ArchivalUrlFormRequestParser.java	2011-09-06 04:20:57 UTC (rev 3533)
@@ -54,12 +54,14 @@
 				accessPoint.translateRequestPathQuery(httpRequest);
 			ArchivalUrl aUrl = new ArchivalUrl(wbRequest);
 			String bestPath = aUrl.toString();
-			if(!bestPath.equals(requestPath)) {
-				String betterURI = (wbRequest.isReplayRequest() ? 
-						accessPoint.getReplayPrefix() : 
-							accessPoint.getQueryPrefix()) 
-						+ bestPath;
-				throw new BetterRequestException(betterURI);
+			if(accessPoint.isForceCleanQueries()) {
+				if(!bestPath.equals(requestPath)) {
+					String betterURI = (wbRequest.isReplayRequest() ? 
+							accessPoint.getReplayPrefix() : 
+								accessPoint.getQueryPrefix()) 
+							+ bestPath;
+					throw new BetterRequestException(betterURI);
+				}
 			}
 		}
 		return wbRequest;

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.