1. awkward yet working support for POST file upload (requires mimetic)

2. support for NOT handling/parsing POSTed data (including application/x-www-form-urlencoded)

4 files changed, 190 insertions, 19 deletions

diff --git a/configure.ac b/configure.ac
index b3141f1..b51d64d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2,3 +2,3 @@ AC_INIT([kingate], [0.0.1], [kingate-bugs@klever.net])
 AC_CONFIG_SRCDIR([include/kingate/cgi_gateway.h])
-AC_CONFIG_HEADER([config.h])
+AC_CONFIG_HEADERS([config.h])
 AM_INIT_AUTOMAKE([dist-bzip2])
@@ -62,2 +62,16 @@ fi
 
+HAVE_MIMETIC=false
+AC_LANG_PUSH(C++)
+  AC_CHECK_LIB([mimetic],[main],[
+   MIMETIC_LIBS=-lmimetic
+   HAVE_MIMETIC=true
+  ]
+ )
+AC_LANG_POP(C++)
+AC_SUBST([MIMETIC_LIBS])
+AC_SUBST([MIMETIC_CFLAGS])
+if ${HAVE_MIMETIC} ; then
+ AC_DEFINE([HAVE_MIMETIC],,[defined in presence of mimetic])
+fi
+
 AC_CONFIG_FILES([
diff --git a/include/kingate/cgi_gateway.h b/include/kingate/cgi_gateway.h
index f20d72b..a26b0ae 100644
--- a/include/kingate/cgi_gateway.h
+++ b/include/kingate/cgi_gateway.h
@@ -45,2 +45,33 @@ namespace kingate {
             /**
+             * Abstract base class for retrieving posted files.
+             */
+            class basic_file_t {
+                public:
+                    /**
+                     * Retrieve file name.
+                     * @return filename
+                     */
+                    virtual const string& filename() const = 0;
+                    /**
+                     * Retrieve file content type.
+                     * @return content type
+                     */
+                    virtual const string& content_type() const = 0;
+                    /**
+                     * Retrieve file contents.
+                     * @return reference to the stream for accessing file content.
+                     */
+                    virtual istream& content() = 0;
+                    virtual ~basic_file_t();
+            };
+            typedef basic_file_t *file_t;
+            /**
+             * The map holding information pertaining to files uploaded via post.
+             */
+            typedef multimap<string,file_t> files_t;
+            /**
+             * Files uploaded via post
+             */
+            files_t files;
+            /**
              * Cookies passed.
@@ -56,3 +87,9 @@ namespace kingate {
              */
-            cgi_gateway(cgi_interface& ci);
+            cgi_gateway(cgi_interface& ci,bool parsebody = true);
+            virtual ~cgi_gateway() throw();
+
+            /**
+             * Parse request body.
+             */
+            void parse_request_body();
 
@@ -138,3 +175,3 @@ namespace kingate {
              * @param n the parameter name.
-             * @return true if yes.
+             * @return the parameter contents.
              * @see exception_notfound.
@@ -142,2 +179,16 @@ namespace kingate {
             const string& get_param(const string& n) const;
+            /**
+             * Check to see whether the file was uploaded in the request body.
+             * @param n the parameter name.
+             * @return true if yes.
+             */
+            bool has_file(const string& n) const;
+            /**
+             * Retrieve the file uploaded in the request body.
+             * @param n the parameter name.
+             * @return the file.
+             * @see exception_notfound.
+             */
+            const file_t get_file(const string& n) const;
+            file_t get_file(const string& n);
 
@@ -179,3 +230,3 @@ namespace kingate {
              */
-            unsigned long cgi_gateway::content_length() const;
+            unsigned long content_length() const;
             /**
diff --git a/kingate.pc.in b/kingate.pc.in
index 671faac..05cfe1d 100644
--- a/kingate.pc.in
+++ b/kingate.pc.in
@@ -9,3 +9,3 @@ Version: @VERSION@
 Requires: konforka
-Libs: -L${libdir} -lkingate
-Cflags: -I${includedir}
+Libs: -L${libdir} -lkingate @MIMETIC_LIBS@
+Cflags: -I${includedir} @MIMETIC_CFLAGS@
diff --git a/src/cgi_gateway.cc b/src/cgi_gateway.cc
index ab48f78..1706679 100644
--- a/src/cgi_gateway.cc
+++ b/src/cgi_gateway.cc
@@ -2,2 +2,3 @@
 #include <ctype.h>
+#include <sstream>
 #include "kingate/cgi_gateway.h"
@@ -5,4 +6,36 @@
 #include "kingate/exception.h"
+#include "config.h"
+#ifdef HAVE_MIMETIC
+# include <mimetic/mimeentity.h>
+# include <mimetic/parser/itparser.h>
+#endif /* HAVE_MIMETIC */
 
 namespace kingate {
+    using mimetic::MimeEntity;
+
+#ifdef HAVE_MIMETIC
+    struct TornMimeEntity : public MimeEntity {
+        typedef istreambuf_iterator<char> it_type;
+        typedef it_type::iterator_category it_cat;
+        struct IParser : public mimetic::IteratorParser<it_type,it_cat> {
+            typedef mimetic::IteratorParser<it_type,it_cat> BT;
+            IParser(MimeEntity& me)
+                : BT::IteratorParser<it_type,it_cat>(me) { }
+            void loadHeader(it_type bit,it_type eit) {
+                m_bit = bit; m_eit = eit;
+                BT::loadHeader();
+            }
+            void loadBody(it_type bit,it_type eit) {
+                m_bit = bit; m_eit = eit;
+                BT::loadBody();
+            }
+        };
+        void load(istream& hs,istream& bs,int mask=0) {
+            IParser prs(*this);
+            prs.iMask(mask);
+            prs.loadHeader(it_type(hs),it_type());
+            prs.loadBody(it_type(bs),it_type());
+        }
+    };
+#endif /* HAVE_MIMETIC */
 
@@ -10,3 +43,18 @@ namespace kingate {
 
-    cgi_gateway::cgi_gateway(cgi_interface& ci)
+    cgi_gateway::basic_file_t::~basic_file_t() { }
+
+    class string_file_t : public cgi_gateway::basic_file_t {
+        public:
+            string _file_name;
+            string _content_type;
+            stringstream _content;
+
+            string_file_t(const string& fn,const string& ct,const string& s)
+                : _file_name(fn), _content_type(ct), _content(s,ios::in) { }
+            const string& filename() const { return _file_name; }
+            const string& content_type() const { return _content_type; }
+            istream& content() { return _content; }
+    };
+
+    cgi_gateway::cgi_gateway(cgi_interface& ci,bool parsebody)
         : iface(ci), b_parsed_content(false) {
@@ -17,14 +65,4 @@ namespace kingate {
             }catch(exception_notfound& enf) { }
-            // Fetch POST content
-            if(!strcasecmp(content_type().c_str(),"application/x-www-form-urlencoded")) {
-                unsigned long cl = content_length();
-                if(cl) {
-                    char * tmp = new char[cl];
-                    iface.in().read(tmp,cl);
-                    string qs(tmp,cl);
-                    delete tmp;
-                    parse_query(qs,post);
-                }
-                b_parsed_content = true;
-            }
+            if(parsebody)
+                parse_request_body();
             // Parse cookies
@@ -35,2 +73,55 @@ namespace kingate {
 
+    cgi_gateway::~cgi_gateway() throw() {
+        for(files_t::iterator i=files.begin();i!=files.end();++i)
+            delete i->second;
+        files.clear();
+    }
+
+    void cgi_gateway::parse_request_body() {
+        if(b_parsed_content)
+            throw konforka::exception(CODEPOINT,"request body is already parsed");
+        // Fetch POST content
+        if(!strncasecmp(
+                    content_type().c_str(),
+                    "application/x-www-form-urlencoded",
+                    sizeof("application/x-www-form-urlencoded")-1) ) {
+            unsigned long cl = content_length();
+            if(cl) {
+                char * tmp = new char[cl];
+                iface.in().read(tmp,cl);
+                string qs(tmp,cl);
+                delete tmp;
+                parse_query(qs,post);
+            }
+            b_parsed_content = true;
+        }
+#ifdef HAVE_MIMETIC
+         else if(!strncasecmp(
+                    content_type().c_str(),
+                    "multipart/form-data",
+                    sizeof("multipart/form-data")-1) ) {
+            stringstream h;
+            h
+                << "Content-Type: " << content_type() << "\r\n"
+                << "Content-Length: " << content_length() << "\r\n\n";
+            TornMimeEntity me;
+            me.load(h,iface.in(),0);
+            mimetic::MimeEntityList& parts = me.body().parts();
+            for(mimetic::MimeEntityList::iterator i=parts.begin();i!=parts.end();++i) {
+                MimeEntity *p = *i;
+                const mimetic::ContentDisposition& cd = p->header().contentDisposition();
+                string n = cd.param("name");
+                string fn = cd.param("filename");
+                if(fn.empty()) {
+                    post.insert(params_t::value_type(n,p->body()));
+                }else{
+                    const mimetic::ContentType& ct = p->header().contentType();
+                    files.insert(files_t::value_type(n,new string_file_t(fn,ct.str(),p->body())));
+                }
+            }
+            b_parsed_content = true;
+        }
+#endif /* HAVE_MIMETIC */
+    }
+
     bool cgi_gateway::has_GET(const string& n) const {
@@ -65,2 +156,17 @@ namespace kingate {
     }
+    bool cgi_gateway::has_file(const string& n) const {
+        return files.find(n) != files.end();
+    }
+    const cgi_gateway::file_t cgi_gateway::get_file(const string& n) const {
+        files_t::const_iterator i = files.find(n);
+        if(i==files.end())
+            throw exception_notfound(CODEPOINT,"no such parameter");
+        return i->second;
+    }
+    cgi_gateway::file_t cgi_gateway::get_file(const string& n) {
+        files_t::const_iterator i = files.find(n);
+        if(i==files.end())
+            throw exception_notfound(CODEPOINT,"no such parameter");
+        return i->second;
+    }