Adhere to Postel's Law.

That is, be liberal when parsing key/value parameters from remote.
Along with configure switch to disable it.

2 files changed, 23 insertions, 0 deletions

diff --git a/configure.ac b/configure.ac
index 8397914..19e7832 100644
--- a/configure.ac
+++ b/configure.ac
@@ -76,4 +76,11 @@ AC_ARG_ENABLE([ssl-verify-peer],
 ${curl_ssl_verify_peer} || AC_DEFINE([DISABLE_CURL_SSL_VERIFYPEER],,[defined if cURL is not to verify cert validity])
 
+postels_law=true
+AC_ARG_ENABLE([postels-law],
+ AC_HELP_STRING([--disable-postels-law],[Be strict, do not adhere to Postel's Law ("be conservative in what you do, be liberal in what you accept from others", RFC 793)]),
+ [ test "${enableval}" = "no" && postels_law=false ]
+)
+$postels_law && AC_DEFINE([POSTELS_LAW],,[defined if we want to adhere to Postel's Law])
+
 AC_CONFIG_FILES([
  Makefile
diff --git a/lib/params.cc b/lib/params.cc
index b181811..ea86d3a 100644
--- a/lib/params.cc
+++ b/lib/params.cc
@@ -5,4 +5,6 @@
 #include <openssl/hmac.h>
 
+#include "config.h"
+
 namespace opkele {
     using namespace std;
@@ -31,4 +33,5 @@ namespace opkele {
 	    if(co==string::npos)
 		break;
+#ifndef POSTELS_LAW
 	    string::size_type nl = kv.find('\n',co+1);
 	    if(nl==string::npos)
@@ -37,4 +40,17 @@ namespace opkele {
 		insert(value_type(kv.substr(p,co-p),kv.substr(co+1,nl-co-1)));
 	    p = nl+1;
+#else /* POSTELS_LAW */
+	    string::size_type lb = kv.find_first_of("\r\n",co+1);
+	    if(lb==string::npos) {
+		insert(value_type(kv.substr(p,co-p),kv.substr(co+1)));
+		break;
+	    }
+	    if(lb>co)
+		insert(value_type(kv.substr(p,co-p),kv.substr(co+1,lb-co-1)));
+	    string::size_type nolb = kv.find_first_not_of("\r\n",lb);
+	    if(nolb==string::npos)
+		break;
+	    p = nolb;
+#endif /* POSTELS_LAW */
 	}
     }