ABP/Android proxy service

src/org/adblockplus/brazil/TransparentProxyHandler.java

+package org.adblockplus.brazil;
+
+import java.io.IOException;
+
+import sunlabs.brazil.server.Handler;
+import sunlabs.brazil.server.Request;
+import sunlabs.brazil.server.Server;
+
+/**
+ * Reconstructs request url to comply with proxy specification if transparent
+ * proxy is used.
+ */
+public class TransparentProxyHandler implements Handler
+{
+
+  @Override
+  public boolean init(Server server, String prefix)
+  {
+    return true;
+  }
+
+  @Override
+  public boolean respond(Request request) throws IOException
+  {
+    if (!request.url.contains("://"))

[Felix Dahlke, 2012/11/08 20:00:43]

Since we're only dealing with HTTP(S), something like this would be less
ambiguous: request.url.matches("^http?s://")

[Andrey Novikov, 2012/11/09 09:23:47]

Why are we? It's universal, it rewrites:
GET /file.name
into:
GET http://host/file.name
It should not do any rewriting in case it will find any schema in URL. I do not
see any reason to break this universalism.

+    {
+      request.url = "http://" + request.headers.get("host") + request.url;
+    }
+    return false;
+  }
+
+}