version 0.1.12
[fms.git] / src / freenet / messagerequester.cpp
index d59a7c0..54b0d26 100644 (file)
@@ -1,6 +1,8 @@
 #include "../../include/freenet/messagerequester.h"\r
 #include "../../include/freenet/messagexml.h"\r
 \r
+#include <algorithm>\r
+\r
 #ifdef XMEM\r
        #include <xmem.h>\r
 #endif\r
@@ -78,6 +80,7 @@ const bool MessageRequester::HandleAllData(FCPMessage &message)
        MessageXML xml;\r
        long identityid;\r
        long index;\r
+       bool inserted=false;\r
 \r
        StringFunctions::Split(message["Identifier"],"|",idparts);\r
        StringFunctions::Convert(message["DataLength"],datalength);\r
@@ -112,6 +115,10 @@ const bool MessageRequester::HandleAllData(FCPMessage &message)
        if(xml.ParseXML(std::string(data.begin(),data.end()))==true)\r
        {\r
                std::vector<std::string> boards=xml.GetBoards();\r
+               if(boards.size()>m_maxboardspermessage)\r
+               {\r
+                       boards.resize(m_maxboardspermessage);\r
+               }\r
 \r
                if(boards.size()<=0)\r
                {\r
@@ -123,6 +130,15 @@ const bool MessageRequester::HandleAllData(FCPMessage &message)
                        m_log->WriteLog(LogFile::LOGLEVEL_ERROR,"MessageRequester::HandleAllData Message XML did not contain a reply board! "+message["Identifier"]);\r
                        return true;\r
                }\r
+               \r
+               // make sure the reply board is on the board list - if not, replace the last element of boardswith the reply board\r
+               if(xml.GetReplyBoard()!="" && std::find(boards.begin(),boards.end(),xml.GetReplyBoard())==boards.end() && boards.size()>0)\r
+               {\r
+                       boards[boards.size()-1]=xml.GetReplyBoard();\r
+               }\r
+\r
+               // TODO make sure domain of message id match 43 characters of public key of identity (remove - and ~) - if not, discard message\r
+               // implement after 0.1.12 is released\r
 \r
                st=m_db->Prepare("INSERT INTO tblMessage(IdentityID,FromName,MessageDate,MessageTime,Subject,MessageUUID,ReplyBoardID,Body) VALUES(?,?,?,?,?,?,?,?);");\r
                st.Bind(0,identityid);\r
@@ -133,32 +149,41 @@ const bool MessageRequester::HandleAllData(FCPMessage &message)
                st.Bind(5,xml.GetMessageID());\r
                st.Bind(6,GetBoardID(xml.GetReplyBoard()));\r
                st.Bind(7,xml.GetBody());\r
-               st.Step(true);\r
+               inserted=st.Step(true);\r
                int messageid=st.GetLastInsertRowID();\r
 \r
-               st=m_db->Prepare("INSERT INTO tblMessageBoard(MessageID,BoardID) VALUES(?,?);");\r
-               for(std::vector<std::string>::iterator i=boards.begin(); i!=boards.end(); i++)\r
+               if(inserted==true)\r
                {\r
-                       st.Bind(0,messageid);\r
-                       st.Bind(1,GetBoardID((*i)));\r
-                       st.Step();\r
-                       st.Reset();\r
-               }\r
-               st.Finalize();\r
 \r
-               st=m_db->Prepare("INSERT INTO tblMessageReplyTo(MessageID,ReplyToMessageUUID,ReplyOrder) VALUES(?,?,?);");\r
-               std::map<long,std::string> replyto=xml.GetInReplyTo();\r
-               for(std::map<long,std::string>::iterator j=replyto.begin(); j!=replyto.end(); j++)\r
+                       st=m_db->Prepare("INSERT INTO tblMessageBoard(MessageID,BoardID) VALUES(?,?);");\r
+                       for(std::vector<std::string>::iterator i=boards.begin(); i!=boards.end(); i++)\r
+                       {\r
+                               st.Bind(0,messageid);\r
+                               st.Bind(1,GetBoardID((*i)));\r
+                               st.Step();\r
+                               st.Reset();\r
+                       }\r
+                       st.Finalize();\r
+\r
+                       st=m_db->Prepare("INSERT INTO tblMessageReplyTo(MessageID,ReplyToMessageUUID,ReplyOrder) VALUES(?,?,?);");\r
+                       std::map<long,std::string> replyto=xml.GetInReplyTo();\r
+                       for(std::map<long,std::string>::iterator j=replyto.begin(); j!=replyto.end(); j++)\r
+                       {\r
+                               st.Bind(0,messageid);\r
+                               st.Bind(1,(*j).second);\r
+                               st.Bind(2,(*j).first);\r
+                               st.Step();\r
+                               st.Reset();\r
+                       }\r
+                       st.Finalize();\r
+\r
+                       m_log->WriteLog(LogFile::LOGLEVEL_DEBUG,"MessageRequester::HandleAllData parsed Message XML file : "+message["Identifier"]);\r
+\r
+               }\r
+               else    // couldn't insert - was already in database\r
                {\r
-                       st.Bind(0,messageid);\r
-                       st.Bind(1,(*j).second);\r
-                       st.Bind(2,(*j).first);\r
-                       st.Step();\r
-                       st.Reset();\r
+                       //m_log->WriteLog(LogFile::LOGLEVEL_ERROR,"MessageRequester::HandleAddData could not insert message into database.  "+message["Identifier"]);\r
                }\r
-               st.Finalize();\r
-\r
-               m_log->WriteLog(LogFile::LOGLEVEL_DEBUG,"MessageRequester::HandleAllData parsed Message XML file : "+message["Identifier"]);\r
 \r
        }\r
        else\r
@@ -209,7 +234,7 @@ void MessageRequester::Initialize()
 {\r
        m_fcpuniquename="MessageRequester";\r
        std::string tempval;\r
-       Option::instance()->Get("MaxMessageRequests",tempval);\r
+       Option::Instance()->Get("MaxMessageRequests",tempval);\r
        StringFunctions::Convert(tempval,m_maxrequests);\r
        if(m_maxrequests<1)\r
        {\r
@@ -220,7 +245,7 @@ void MessageRequester::Initialize()
        {\r
                m_log->WriteLog(LogFile::LOGLEVEL_WARNING,"Option MaxMessageRequests is currently set at "+tempval+".  This value might be incorrectly configured.");\r
        }\r
-       Option::instance()->Get("MessageDownloadMaxDaysBackward",tempval);\r
+       Option::Instance()->Get("MessageDownloadMaxDaysBackward",tempval);\r
        StringFunctions::Convert(tempval,m_maxdaysbackward);\r
        if(m_maxdaysbackward<0)\r
        {\r
@@ -231,6 +256,28 @@ void MessageRequester::Initialize()
        {\r
                m_log->WriteLog(LogFile::LOGLEVEL_WARNING,"Option MessageDownloadMaxDaysBackward is currently set at "+tempval+".  This value might be incorrectly configured.");\r
        }\r
+       Option::Instance()->Get("MaxPeerMessagesPerDay",tempval);\r
+       StringFunctions::Convert(tempval,m_maxpeermessages);\r
+       if(m_maxpeermessages<1)\r
+       {\r
+               m_maxpeermessages=1;\r
+               m_log->WriteLog(LogFile::LOGLEVEL_ERROR,"Option MaxPeerMessagesPerDay is currently set at "+tempval+".  It must be 1 or greater.");\r
+       }\r
+       if(m_maxpeermessages<20 || m_maxpeermessages>1000)\r
+       {\r
+               m_log->WriteLog(LogFile::LOGLEVEL_WARNING,"Option MaxPeerMessagesPerDay is currently set at "+tempval+".  This value might be incorrectly configured.  The suggested value is 200.");\r
+       }\r
+       Option::Instance()->Get("MaxBoardsPerMessage",tempval);\r
+       StringFunctions::Convert(tempval,m_maxboardspermessage);\r
+       if(m_maxboardspermessage<1)\r
+       {\r
+               m_maxboardspermessage=1;\r
+               m_log->WriteLog(LogFile::LOGLEVEL_ERROR,"Option MaxBoardsPerMessage is currently set at "+tempval+".  It must be 1 or greater.");\r
+       }\r
+       if(m_maxboardspermessage>20)\r
+       {\r
+               m_log->WriteLog(LogFile::LOGLEVEL_WARNING,"Option MaxBoardsPerMessage is currently set at "+tempval+".  This value might be incorrectly configured.");\r
+       }\r
 }\r
 \r
 void MessageRequester::PopulateIDList()\r
@@ -239,21 +286,39 @@ void MessageRequester::PopulateIDList()
        std::string val1;\r
        std::string val2;\r
        std::string val3;\r
+       std::string sql;\r
+       long requestindex;\r
 \r
        date.SetToGMTime();\r
        date.Add(0,0,0,-m_maxdaysbackward);\r
 \r
-       SQLite3DB::Statement st=m_db->Prepare("SELECT tblIdentity.IdentityID,Day,RequestIndex FROM tblMessageRequests INNER JOIN tblIdentity ON tblMessageRequests.IdentityID=tblIdentity.IdentityID WHERE tblIdentity.LocalMessageTrust>=(SELECT OptionValue FROM tblOption WHERE Option='MinLocalMessageTrust') AND FromMessageList='true' AND Found='false' AND Day>='"+date.Format("%Y-%m-%d")+"';");\r
+       sql="SELECT tblIdentity.IdentityID,Day,RequestIndex ";\r
+       sql+="FROM tblMessageRequests INNER JOIN tblIdentity ON tblMessageRequests.IdentityID=tblIdentity.IdentityID ";\r
+       sql+="WHERE tblIdentity.LocalMessageTrust>=(SELECT OptionValue FROM tblOption WHERE Option='MinLocalMessageTrust') AND FromMessageList='true' AND Found='false' AND Day>='"+date.Format("%Y-%m-%d")+"' ";\r
+       sql+="AND (tblIdentity.PeerMessageTrust IS NULL OR tblIdentity.PeerMessageTrust>=(SELECT OptionValue FROM tblOption WHERE Option='MinPeerMessageTrust')) ";\r
+       sql+=";";\r
+\r
+       SQLite3DB::Statement st=m_db->Prepare(sql);\r
        st.Step();\r
 \r
+       m_ids.clear();\r
+\r
        while(st.RowReturned())\r
        {\r
                st.ResultText(0,val1);\r
                st.ResultText(1,val2);\r
                st.ResultText(2,val3);\r
-               if(m_ids.find(val1+"*"+val2+"*"+val3)==m_ids.end())\r
+\r
+               requestindex=0;\r
+               StringFunctions::Convert(val3,requestindex);\r
+\r
+               // only continue if index is < max messages we will accept from a peer\r
+               if(requestindex<m_maxpeermessages)\r
                {\r
-                       m_ids[val1+"*"+val2+"*"+val3]=false;\r
+                       if(m_ids.find(val1+"*"+val2+"*"+val3)==m_ids.end())\r
+                       {\r
+                               m_ids[val1+"*"+val2+"*"+val3]=false;\r
+                       }\r
                }\r
                st.Step();\r
        }\r
@@ -288,12 +353,15 @@ void MessageRequester::StartRequest(const std::string &requestid)
                message["Identifier"]=m_fcpuniquename+"|"+requestid+"|"+parts[0]+"|"+parts[1]+"|"+parts[2]+"|"+message["URI"];\r
                message["ReturnType"]="direct";\r
                message["MaxSize"]="1000000";           // 1 MB\r
+               message["MaxRetries"]="-1";                     // use new ULPR since we are fairly sure message exists since the author says it does\r
 \r
                m_fcp->SendMessage(message);\r
 \r
                m_requesting.push_back(requestid);\r
+\r
+               m_log->WriteLog(LogFile::LOGLEVEL_DEBUG,"MessageRequester::StartRequest requesting "+message["Identifier"]);\r
        }\r
        \r
        m_ids[requestid]=true;\r
 \r
-}
\ No newline at end of file
+}\r