From: Adam Dickmeiss <adam@indexdata.dk>
Date: Mon, 19 Jun 2006 13:08:00 +0000 (+0000)
Subject: Log filter can be configured to write to a given file using 'filename'
X-Git-Tag: METAPROXY.1.0.6~21
X-Git-Url: http://jsfdemo.indexdata.com/cgi-bin?a=commitdiff_plain;h=8db6e2826e50fb4d3f8e6a1287b06357409cf88b;p=metaproxy-moved-to-github.git

Log filter can be configured to write to a given file using 'filename'
element. Also information category can be specified with 'category'.
If filename is omitted, the log filter writes to metaproxy.log.
See etc/config5.xml for example.
---

diff --git a/NEWS b/NEWS
index de814da..807b692 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,8 @@
+Log filter can be configured to write to a given file using 'filename'
+element. Also information category can be specified with 'category'.
+If filename is omitted, the log filter writes to metaproxy.log.
+See etc/config5.xml for example.
+
 --- 1.0.5 2006/05/16
 
 Fixed compilation on Windows.
diff --git a/doc/log.xml b/doc/log.xml
index cfee1af..c6e9907 100644
--- a/doc/log.xml
+++ b/doc/log.xml
@@ -2,7 +2,7 @@
     "http://www.oasis-open.org/docbook/xml/4.1/docbookx.dtd" [
  <!ENTITY copyright SYSTEM "copyright.xml">
 ]>
-<!-- $Id: log.xml,v 1.4 2006-04-25 12:48:52 marc Exp $ -->
+<!-- $Id: log.xml,v 1.5 2006-06-19 13:08:00 adam Exp $ -->
 <refentry>
  <refmeta>
   <refentrytitle>log</refentrytitle>
@@ -27,6 +27,7 @@
    <screen><![CDATA[
     <filter type="log">
      <message>B</message>
+     <filename>logs/etaproxy.log</filename>
     </filter>
 ]]>
    </screen>
diff --git a/src/filter_log.cpp b/src/filter_log.cpp
index c4ec3a3..fc457bb 100644
--- a/src/filter_log.cpp
+++ b/src/filter_log.cpp
@@ -1,4 +1,4 @@
-/* $Id: filter_log.cpp,v 1.18 2006-06-10 14:29:12 adam Exp $
+/* $Id: filter_log.cpp,v 1.19 2006-06-19 13:08:00 adam Exp $
    Copyright (c) 2005-2006, Index Data.
 
    See the LICENSE file for details
@@ -13,23 +13,49 @@
 #include <boost/date_time/posix_time/posix_time.hpp>
 
 #include "util.hpp"
+#include "xmlutil.hpp"
 #include "filter_log.hpp"
 
+#include <fstream>
 #include <yaz/zgdu.h>
 
 namespace mp = metaproxy_1;
-namespace yf = mp::filter;
+namespace yf = metaproxy_1::filter;
 
 namespace metaproxy_1 {
     namespace filter {
+        class Log::LFile {
+        public:
+            boost::mutex m_mutex;
+            std::string m_fname;
+            std::ofstream out;
+            LFile(std::string fname);
+        };
+        typedef boost::shared_ptr<Log::LFile> LFilePtr;
         class Log::Rep {
             friend class Log;
-            boost::mutex m_log_mutex;
+            Rep();
+            void openfile(const std::string &fname);
             std::string m_msg;
+            LFilePtr m_file;
+            bool m_req_apdu;
+            bool m_res_apdu;
+            bool m_req_session;
+            bool m_res_session;
         };
+        // Only used during configure stage (no threading)
+        static std::list<LFilePtr> filter_log_files;
     }
 }
 
+yf::Log::Rep::Rep()
+{
+    m_req_apdu = true;
+    m_res_apdu = true;
+    m_req_session = true;
+    m_res_session = true;
+}
+
 yf::Log::Log(const std::string &x) : m_p(new Rep)
 {
     m_p->m_msg = x;
@@ -41,6 +67,12 @@ yf::Log::Log() : m_p(new Rep)
 
 yf::Log::~Log() {}
 
+void stream_write(ODR o, void *handle, int type, const char *buf, int len)
+{
+    yf::Log::LFile *lfile = (yf::Log::LFile*) handle;
+    lfile->out.write(buf, len);
+}
+
 void yf::Log::process(mp::Package &package) const
 {
     Z_GDU *gdu;
@@ -51,18 +83,27 @@ void yf::Log::process(mp::Package &package) const
 
     // scope for locking Ostream 
     { 
-        boost::mutex::scoped_lock scoped_lock(m_p->m_log_mutex);
-        std::cout << receive_time << " " << m_p->m_msg;
-        std::cout << " request id=" << package.session().id();
-        std::cout << " close=" 
-                  << (package.session().is_closed() ? "yes" : "no")
-                  << "\n";
-        gdu = package.request().get();
-        if (gdu)
+        boost::mutex::scoped_lock scoped_lock(m_p->m_file->m_mutex);
+        
+        if (m_p->m_req_session)
         {
-            mp::odr odr(ODR_PRINT);
-            z_GDU(odr, &gdu, 0, 0);
+            m_p->m_file->out << receive_time << " " << m_p->m_msg;
+            m_p->m_file->out << " request id=" << package.session().id();
+            m_p->m_file->out << " close=" 
+                             << (package.session().is_closed() ? "yes" : "no")
+                             << "\n";
         }
+        if (m_p->m_req_apdu)
+        {
+            gdu = package.request().get();
+            if (gdu)
+            {
+                mp::odr odr(ODR_PRINT);
+                odr_set_stream(odr, m_p->m_file.get(), stream_write, 0);
+                z_GDU(odr, &gdu, 0, 0);
+            }
+        }
+        m_p->m_file->out.flush();
     }
 
     // unlocked during move
@@ -76,23 +117,49 @@ void yf::Log::process(mp::Package &package) const
 
     // scope for locking Ostream 
     { 
-        boost::mutex::scoped_lock scoped_lock(m_p->m_log_mutex);
-        std::cout << send_time << " " << m_p->m_msg;
-        std::cout << " response id=" << package.session().id();
-        std::cout << " close=" 
-                  << (package.session().is_closed() ? "yes " : "no ")
-                  << "duration=" << duration      
-                  << "\n";
-            //<< "duration=" << duration.total_seconds() 
-            //    << "." << duration.fractional_seconds()
-            //      << "\n";
-        gdu = package.response().get();
-        if (gdu)
+        boost::mutex::scoped_lock scoped_lock(m_p->m_file->m_mutex);
+        if (m_p->m_res_session)
+        {
+            m_p->m_file->out << send_time << " " << m_p->m_msg;
+            m_p->m_file->out << " response id=" << package.session().id();
+            m_p->m_file->out << " close=" 
+                             << (package.session().is_closed() ? "yes " : "no ")
+                             << "duration=" << duration      
+                             << "\n";
+        }
+        if (m_p->m_res_apdu)
+        {
+            gdu = package.response().get();
+            if (gdu)
+            {
+                mp::odr odr(ODR_PRINT);
+                odr_set_stream(odr, m_p->m_file.get(), stream_write, 0);
+                z_GDU(odr, &gdu, 0, 0);
+            }
+        }
+        m_p->m_file->out.flush();
+    }
+}
+
+yf::Log::LFile::LFile(std::string fname) : m_fname(fname)
+{
+    out.open(fname.c_str());
+}
+
+void yf::Log::Rep::openfile(const std::string &fname)
+{
+    std::list<LFilePtr>::const_iterator it = filter_log_files.begin();
+    for (; it != filter_log_files.end(); it++)
+    {
+        if ((*it)->m_fname == fname)
         {
-            mp::odr odr(ODR_PRINT);
-            z_GDU(odr, &gdu, 0, 0);
+            m_file = *it;
+            return;
         }
     }
+    LFilePtr newfile(new LFile(fname));
+    filter_log_files.push_back(newfile);
+    m_file = newfile;
 }
 
 void yf::Log::configure(const xmlNode *ptr)
@@ -103,6 +170,34 @@ void yf::Log::configure(const xmlNode *ptr)
             continue;
         if (!strcmp((const char *) ptr->name, "message"))
             m_p->m_msg = mp::xml::get_text(ptr);
+        else if (!strcmp((const char *) ptr->name, "filename"))
+        {
+            std::string fname = mp::xml::get_text(ptr);
+            m_p->openfile(fname);
+        }
+        else if (!strcmp((const char *) ptr->name, "category"))
+        {
+            const struct _xmlAttr *attr;
+            for (attr = ptr->properties; attr; attr = attr->next)
+            {
+                if (!strcmp((const char *) attr->name, "request-apdu"))
+                    m_p->m_req_apdu = mp::xml::get_bool(attr->children, true);
+                else if (!strcmp((const char *) attr->name, "response-apdu"))
+                    m_p->m_res_apdu = mp::xml::get_bool(attr->children, true);
+                else if (!strcmp((const char *) attr->name,
+                                 "request-session"))
+                    m_p->m_req_session = 
+                        mp::xml::get_bool(attr->children, true);
+                else if (!strcmp((const char *) attr->name, 
+                                 "response-session"))
+                    m_p->m_res_session = 
+                        mp::xml::get_bool(attr->children, true);
+                else
+                    throw mp::filter::FilterException(
+                        "Bad attribute " + std::string((const char *)
+                                                       attr->name));
+            }
+        }
         else
         {
             throw mp::filter::FilterException("Bad element " 
@@ -110,6 +205,8 @@ void yf::Log::configure(const xmlNode *ptr)
                                                              ptr->name));
         }
     }
+    if (!m_p->m_file)
+        m_p->openfile("metaproxy.log");
 }
 
 static mp::filter::Base* filter_creator()
@@ -125,7 +222,6 @@ extern "C" {
     };
 }
 
-
 /*
  * Local variables:
  * c-basic-offset: 4
diff --git a/src/filter_log.hpp b/src/filter_log.hpp
index 6e048b6..dc18af6 100644
--- a/src/filter_log.hpp
+++ b/src/filter_log.hpp
@@ -1,4 +1,4 @@
-/* $Id: filter_log.hpp,v 1.15 2006-06-10 14:29:12 adam Exp $
+/* $Id: filter_log.hpp,v 1.16 2006-06-19 13:08:00 adam Exp $
    Copyright (c) 2005-2006, Index Data.
 
    See the LICENSE file for details
@@ -8,6 +8,7 @@
 #define FILTER_LOG_HPP
 
 #include <boost/scoped_ptr.hpp>
+#include <boost/shared_ptr.hpp>
 
 #include "filter.hpp"
 
@@ -22,6 +23,7 @@ namespace metaproxy_1 {
             ~Log();
             void process(metaproxy_1::Package & package) const;
             void configure(const xmlNode * ptr);
+            class LFile;
         };
     }
 }
diff --git a/src/xmlutil.cpp b/src/xmlutil.cpp
index d85ba16..0a333ea 100644
--- a/src/xmlutil.cpp
+++ b/src/xmlutil.cpp
@@ -1,9 +1,10 @@
-/* $Id: xmlutil.cpp,v 1.7 2006-06-10 14:29:13 adam Exp $
+/* $Id: xmlutil.cpp,v 1.8 2006-06-19 13:08:00 adam Exp $
    Copyright (c) 2005-2006, Index Data.
 
    See the LICENSE file for details
  */
 
+#include <string.h>
 #include "xmlutil.hpp"
 
 namespace mp = metaproxy_1;
@@ -19,6 +20,17 @@ std::string mp_xml::get_text(const xmlNode *ptr)
     return c;
 }
 
+bool mp_xml::get_bool(const xmlNode *ptr, bool default_value)
+{
+    if (ptr && ptr->type == XML_TEXT_NODE && ptr->content)
+    {
+        if (!strcmp((const char *) ptr->content, "true"))
+            return true;
+        else
+            return false;
+    }
+    return default_value;
+}
 
 bool mp_xml::is_element(const xmlNode *ptr, 
                           const std::string &ns,
diff --git a/src/xmlutil.hpp b/src/xmlutil.hpp
index c5680ed..27d95af 100644
--- a/src/xmlutil.hpp
+++ b/src/xmlutil.hpp
@@ -1,4 +1,4 @@
-/* $Id: xmlutil.hpp,v 1.6 2006-06-10 14:29:13 adam Exp $
+/* $Id: xmlutil.hpp,v 1.7 2006-06-19 13:08:00 adam Exp $
    Copyright (c) 2005-2006, Index Data.
 
    See the LICENSE file for details
@@ -14,6 +14,7 @@
 namespace metaproxy_1 {
     namespace xml {
         std::string get_text(const xmlNode *ptr);
+        bool get_bool(const xmlNode *ptr, bool default_value);
         bool is_element(const xmlNode *ptr, 
                         const std::string &ns,
                         const std::string &name);