From: Adam Dickmeiss <adam@indexdata.dk>
Date: Tue, 11 Jun 2013 13:20:17 +0000 (+0200)
Subject: sru_z3950: separate x-client prefix for each dbargs
X-Git-Tag: v1.3.57~6
X-Git-Url: http://sru.miketaylor.org.uk/?a=commitdiff_plain;h=fc881acd97f7302976646f7d197d45d86469e243;p=metaproxy-moved-to-github.git

sru_z3950: separate x-client prefix for each dbargs

Eg x-client-user=kurt rather than x-dbargs=user&3Dkurt .
---

diff --git a/src/filter_sru_to_z3950.cpp b/src/filter_sru_to_z3950.cpp
index 44b72bf..ff15d8e 100644
--- a/src/filter_sru_to_z3950.cpp
+++ b/src/filter_sru_to_z3950.cpp
@@ -94,7 +94,7 @@ namespace metaproxy_1 {
                 Z_SRW_PDU *sru_pdu_res,
                 Z_SRW_searchRetrieveRequest const *sr_req,
                 std::string zurl,
-                const char *db_append
+                std::string db_append
                 ) const;
 
             bool z3950_present_request(
@@ -225,7 +225,7 @@ void yf::SRUtoZ3950::Impl::sru(mp::Package &package, Z_GDU *zgdu_req)
 
     bool enable_package_log = false;
     std::string zurl;
-    const char *dbargs = 0;
+    std::string dbargs;
     Z_SRW_extra_arg *arg;
 
     for ( arg = sru_pdu_req->extra_args; arg; arg = arg->next)
@@ -249,9 +249,13 @@ void yf::SRUtoZ3950::Impl::sru(mp::Package &package, Z_GDU *zgdu_req)
                 package.log_enable();
             }
         }
-        else if (!strcmp(arg->name, "x-dbargs"))
+        else if (!strncmp(arg->name, "x-client-", 9) && arg->value)
         {
-            dbargs = arg->value;
+            if (dbargs.length())
+                dbargs += '&';
+            dbargs += mp_util::uri_encode(arg->name + 9);
+            dbargs += '=';
+            dbargs += mp_util::uri_encode(arg->value);
         }
 
     assert(sru_pdu_req);
@@ -557,7 +561,7 @@ bool yf::SRUtoZ3950::Impl::z3950_search_request(mp::Package &package,
                                                 Z_SRW_searchRetrieveRequest
                                                 const *sr_req,
                                                 std::string zurl,
-                                                const char *dbappend) const
+                                                std::string dbappend) const
 {
 
     assert(sru_pdu_res->u.response);
@@ -583,7 +587,7 @@ bool yf::SRUtoZ3950::Impl::z3950_search_request(mp::Package &package,
         else
             db = "Default";
 
-        if (dbappend)
+        if (dbappend.length())
         {
             db += ",";
             db += dbappend;