Rewrite body too

[metaproxy-moved-to-github.git] / src / test_filter_rewrite.cpp

diff --git a/src/test_filter_rewrite.cpp b/src/test_filter_rewrite.cpp
index 67755dd..eee048d 100644 (file)
--- a/src/test_filter_rewrite.cpp
+++ b/src/test_filter_rewrite.cpp
@@ -42,7 +42,8 @@ typedef spair_vec::iterator spv_iter;
 
 class FilterHeaderRewrite: public mp::filter::Base {
 public:
-    void process(mp::Package & package) const {
+    void process(mp::Package & package) const 
+    {
         Z_GDU *gdu = package.request().get();
         //map of request/response vars
         std::map<std::string, std::string> vars;
@@ -51,64 +52,96 @@ public:
         {
             Z_HTTP_Request *hreq = gdu->u.HTTP_Request;
             mp::odr o;
-            //rewrite the request line
-            std::string path;
-            if (strstr(hreq->path, "http://") == hreq->path)
-            {
-                std::cout << "Path in the method line is absolute, " 
-                    "possibly a proxy request\n";
-                path += hreq->path;
-            }
-            else
-            {
-                //TODO what about proto
-               path += z_HTTP_header_lookup(hreq->headers, "Host");
-               path += hreq->path; 
-            }
-            std::cout << "Proxy request URL is " << path << std::endl;
-            std::string npath = 
-                test_patterns(vars, path, req_uri_pats, req_groups_bynum);
-            std::cout << "Resp request URL is " << npath << std::endl;
-            if (!npath.empty())
-                hreq->path = odr_strdup(o, npath.c_str());
             std::cout << ">> Request headers" << std::endl;
-            //iterate headers
-            for (Z_HTTP_Header *header = hreq->headers;
-                    header != 0; 
-                    header = header->next) 
-            {
-                std::cout << header->name << ": " << header->value << std::endl;
-                std::string out = test_patterns(vars, 
-                        std::string(header->value), 
-                        req_uri_pats, req_groups_bynum);
-                if (!out.empty())
-                    header->value = odr_strdup(o, out.c_str());
-            }
+            rewrite_reqline(o, hreq, vars);
+            rewrite_headers(o, hreq->headers, vars);
+            rewrite_body(o, &hreq->content_buf, &hreq->content_len, vars);
             package.request() = gdu;
         }
         package.move();
         gdu = package.response().get();
         if (gdu && gdu->which == Z_GDU_HTTP_Response)
         {
-            Z_HTTP_Response *hr = gdu->u.HTTP_Response;
-            std::cout << "Response " << hr->code;
+            Z_HTTP_Response *hres = gdu->u.HTTP_Response;
+            std::cout << "Response " << hres->code;
             std::cout << "<< Respose headers" << std::endl;
             mp::odr o;
-            //iterate headers
-            for (Z_HTTP_Header *header = hr->headers;
-                    header != 0; 
-                    header = header->next) 
+            rewrite_headers(o, hres->headers, vars);
+            rewrite_body(o, &hres->content_buf, &hres->content_len, vars);
+            package.response() = gdu;
+        }
+    }
+
+    void rewrite_reqline (mp::odr & o, Z_HTTP_Request *hreq,
+            std::map<std::string, std::string> & vars) const 
+    {
+        //rewrite the request line
+        std::string path;
+        if (strstr(hreq->path, "http://") == hreq->path)
+        {
+            std::cout << "Path in the method line is absolute, " 
+                "possibly a proxy request\n";
+            path += hreq->path;
+        }
+        else
+        {
+            //TODO what about proto
+            path += z_HTTP_header_lookup(hreq->headers, "Host");
+            path += hreq->path; 
+        }
+        std::cout << "Proxy request URL is " << path << std::endl;
+        std::string npath = 
+            test_patterns(vars, path, req_uri_pats, req_groups_bynum);
+        std::cout << "Resp request URL is " << npath << std::endl;
+        if (!npath.empty())
+            hreq->path = odr_strdup(o, npath.c_str());
+    }
+ 
+    void rewrite_headers (mp::odr & o, Z_HTTP_Header *headers,
+            std::map<std::string, std::string> & vars) const 
+    {
+        for (Z_HTTP_Header *header = headers;
+                header != 0; 
+                header = header->next) 
+        {
+            std::string sheader(header->name);
+            sheader += ": ";
+            sheader += header->value;
+            std::cout << header->name << ": " << header->value << std::endl;
+            std::string out = test_patterns(vars, 
+                    sheader, 
+                    req_uri_pats, req_groups_bynum);
+            if (!out.empty()) 
             {
-                std::cout << header->name << ": " << header->value << std::endl;
-                std::string out = test_patterns(vars,
-                        std::string(header->value), 
-                        res_uri_pats, res_groups_bynum); 
-                if (!out.empty())
-                    header->value = odr_strdup(o, out.c_str());
+                size_t pos = out.find(": ");
+                if (pos == std::string::npos)
+                {
+                    std::cout << "Header malformed during rewrite, ignoring";
+                    continue;
+                }
+                header->name = odr_strdup(o, out.substr(0, pos).c_str());
+                header->value = odr_strdup(o, out.substr(pos+2, 
+                            std::string::npos).c_str());
             }
-            package.response() = gdu;
         }
-    };
+    }
+
+    void rewrite_body (mp::odr & o, char **content_buf, int *content_len,
+            std::map<std::string, std::string> & vars) const 
+    {
+        if (*content_buf)
+        {
+            std::string body(*content_buf);
+            std::string nbody = 
+                test_patterns(vars, body, req_uri_pats, req_groups_bynum);
+            if (!nbody.empty())
+            {
+                *content_buf = odr_strdup(o, nbody.c_str());
+                *content_len = nbody.size();
+            }
+        }
+    }
+
 
     void configure(const xmlNode* ptr, bool test_only, const char *path) {};
 
@@ -304,7 +337,6 @@ public:
     };
 
 private:
-    std::map<std::string, std::string> vars;
     spair_vec req_uri_pats;
     spair_vec res_uri_pats;
     std::vector<std::map<int, std::string> > req_groups_bynum;
@@ -335,12 +367,12 @@ BOOST_AUTO_TEST_CASE( test_filter_rewrite_2 )
         spair_vec vec_req;
         vec_req.push_back(std::make_pair(
         "(?<proto>http\\:\\/\\/s?)(?<pxhost>[^\\/?#]+)\\/(?<pxpath>[^\\/]+)"
-        "\\/(?<target>.+)",
-        "${proto}${target}"
+        "\\/(?<host>[^\\/]+)(?<path>.*)",
+        "${proto}${host}${path}"
         ));
         vec_req.push_back(std::make_pair(
-        "proxyhost",
-        "localhost"
+        "(?:Host\\: )(.*)",
+        "Host: localhost"
         ));
 
         spair_vec vec_res;