X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=citadel%2Fmodules%2Fsmtp%2Fserv_smtpqueue.c;h=5b471127ed176d4060d13c7f2eac0beda927c50b;hb=c1c91208257a253e65275545aea79bba4699ba09;hp=72e5bc231d60a0b1b47c7dd00239cdb4cebb765e;hpb=d3a29d82d6492191ae8208617e8ac1f088bcf14a;p=citadel.git diff --git a/citadel/modules/smtp/serv_smtpqueue.c b/citadel/modules/smtp/serv_smtpqueue.c index 72e5bc231..5b471127e 100644 --- a/citadel/modules/smtp/serv_smtpqueue.c +++ b/citadel/modules/smtp/serv_smtpqueue.c @@ -86,6 +86,7 @@ #include "ctdl_module.h" #include "smtpqueue.h" +#include "smtp_clienthandlers.h" #include "event_client.h" @@ -93,11 +94,24 @@ struct CitContext smtp_queue_CC; pthread_mutex_t ActiveQItemsLock; HashList *ActiveQItems = NULL; HashList *QItemHandlers = NULL; +const unsigned short DefaultMXPort = 25; +int max_sessions_for_outbound_smtp = 500; /* how many sessions might be active till we stop adding more smtp jobs */ +int ndelay_count = 50; /* every n queued messages we will sleep... */ +int delay_msec = 5000; /* this many seconds. */ static const long MaxRetry = SMTP_RETRY_INTERVAL * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2 * 2; int MsgCount = 0; int run_queue_now = 0; /* Set to 1 to ignore SMTP send retry times */ +void RegisterQItemHandler(const char *Key, long Len, QItemHandler H) +{ + QItemHandlerStruct *HS = (QItemHandlerStruct*)malloc(sizeof(QItemHandlerStruct)); + HS->H = H; + Put(QItemHandlers, Key, Len, HS, NULL); +} + + + void smtp_try_one_queue_entry(OneQueItem *MyQItem, MailQEntry *MyQEntry, StrBuf *MsgText, @@ -129,6 +143,22 @@ int DecreaseQReference(OneQueItem *MyQItem) return IDestructQueItem; } +void DecreaseShutdownDeliveries(OneQueItem *MyQItem) +{ + pthread_mutex_lock(&ActiveQItemsLock); + MyQItem->NotYetShutdownDeliveries--; + pthread_mutex_unlock(&ActiveQItemsLock); +} + +int GetShutdownDeliveries(OneQueItem *MyQItem) +{ + int DestructNow; + + pthread_mutex_lock(&ActiveQItemsLock); + DestructNow = MyQItem->ActiveDeliveries == 0; + pthread_mutex_unlock(&ActiveQItemsLock); + return DestructNow; +} void RemoveQItem(OneQueItem *MyQItem) { long len; @@ -142,13 +172,13 @@ void RemoveQItem(OneQueItem *MyQItem) DeleteEntryFromHash(ActiveQItems, It); else { - syslog(LOG_WARNING, - "SMTP cleanup: unable to find QItem with ID[%ld]", - MyQItem->MessageID); + SMTPC_syslog(LOG_WARNING, + "unable to find QItem with ID[%ld]", + MyQItem->MessageID); while (GetNextHashPos(ActiveQItems, It, &len, &Key, &VData)) - syslog(LOG_WARNING, - "SMTP cleanup: have_: ID[%ld]", - ((OneQueItem *)VData)->MessageID); + SMTPC_syslog(LOG_WARNING, + "have_: ID[%ld]", + ((OneQueItem *)VData)->MessageID); } pthread_mutex_unlock(&ActiveQItemsLock); DeleteHashPos(&It); @@ -158,16 +188,28 @@ void RemoveQItem(OneQueItem *MyQItem) void FreeMailQEntry(void *qv) { MailQEntry *Q = qv; +/* + SMTPC_syslog(LOG_DEBUG, "---------------%s--------------", __FUNCTION__); + cit_backtrace(); +*/ FreeStrBuf(&Q->Recipient); FreeStrBuf(&Q->StatusMessage); + + memset(Q, 0, sizeof(MailQEntry)); free(Q); } void FreeQueItem(OneQueItem **Item) { +/* + SMTPC_syslog(LOG_DEBUG, "---------------%s--------------", __FUNCTION__); + cit_backtrace(); +*/ DeleteHash(&(*Item)->MailQEntries); FreeStrBuf(&(*Item)->EnvelopeFrom); FreeStrBuf(&(*Item)->BounceTo); + FreeStrBuf(&(*Item)->SenderRoom); FreeURL(&(*Item)->URL); + memset(*Item, 0, sizeof(OneQueItem)); free(*Item); Item = NULL; } @@ -181,7 +223,30 @@ void HFreeQueItem(void *Item) * - 3/4 (transient errors * were experienced and it's time to try again) */ -int CountActiveQueueEntries(OneQueItem *MyQItem) +int CheckQEntryActive(MailQEntry *ThisItem) +{ + if ((ThisItem->Status == 0) || + (ThisItem->Status == 3) || + (ThisItem->Status == 4)) + { + return 1; + } + else + return 0; +} +int CheckQEntryIsBounce(MailQEntry *ThisItem) +{ + if ((ThisItem->Status == 3) || + (ThisItem->Status == 4) || + (ThisItem->Status == 5)) + { + return 1; + } + else + return 0; +} + +int CountActiveQueueEntries(OneQueItem *MyQItem, int before) { HashPos *It; long len; @@ -193,16 +258,20 @@ int CountActiveQueueEntries(OneQueItem *MyQItem) It = GetNewHashPos(MyQItem->MailQEntries, 0); while (GetNextHashPos(MyQItem->MailQEntries, It, &len, &Key, &vQE)) { + int Active; MailQEntry *ThisItem = vQE; - if ((ThisItem->Status == 0) || - (ThisItem->Status == 3) || - (ThisItem->Status == 4)) + + if (CheckQEntryActive(ThisItem)) { ActiveDeliveries++; - ThisItem->Active = 1; + Active = 1; } else - ThisItem->Active = 0; + Active = 0; + if (before) + ThisItem->Active = Active; + else + ThisItem->StillActive = Active; } DeleteHashPos(&It); return ActiveDeliveries; @@ -233,9 +302,9 @@ OneQueItem *DeserializeQueueItem(StrBuf *RawQItem, long QueMsgID) StrBufExtract_NextToken(Token, Line, &pItemPart, '|'); if (GetHash(QItemHandlers, SKEY(Token), &vHandler)) { - QItemHandler H; - H = (QItemHandler) vHandler; - H(Item, Line, &pItemPart); + QItemHandlerStruct *HS; + HS = (QItemHandlerStruct*) vHandler; + HS->H(Item, Line, &pItemPart); } } FreeStrBuf(&Line); @@ -293,24 +362,24 @@ StrBuf *SerializeQueueItem(OneQueItem *MyQItem) StrBufAppendBuf(QMessage, MyQItem->EnvelopeFrom, 0); } + if (StrLength(MyQItem->SenderRoom) > 0) { + StrBufAppendBufPlain(QMessage, HKEY("\nsource_room|"), 0); + StrBufAppendBuf(QMessage, MyQItem->SenderRoom, 0); + } + StrBufAppendBufPlain(QMessage, HKEY("\nretry|"), 0); StrBufAppendPrintf(QMessage, "%ld", MyQItem->Retry); StrBufAppendBufPlain(QMessage, HKEY("\nattempted|"), 0); StrBufAppendPrintf(QMessage, "%ld", - MyQItem->ReattemptWhen); + time(NULL) /*ctdl_ev_now()*/ + MyQItem->Retry); It = GetNewHashPos(MyQItem->MailQEntries, 0); while (GetNextHashPos(MyQItem->MailQEntries, It, &len, &Key, &vQE)) { MailQEntry *ThisItem = vQE; - if (!ThisItem->Active) - { - /* skip already sent ones from the spoolfile. */ - continue; - } StrBufAppendBufPlain(QMessage, HKEY("\nremote|"), 0); StrBufAppendBuf(QMessage, ThisItem->Recipient, 0); StrBufAppendBufPlain(QMessage, HKEY("|"), 0); @@ -334,7 +403,8 @@ void NewMailQEntry(OneQueItem *Item) if (Item->MailQEntries == NULL) Item->MailQEntries = NewHash(1, Flathash); - Item->Current->StatusMessage = NewStrBuf(); + /* alocate big buffer so we won't get problems reallocating later. */ + Item->Current->StatusMessage = NewStrBufPlain(NULL, SIZ); Item->Current->n = GetCount(Item->MailQEntries); Put(Item->MailQEntries, IKEY(Item->Current->n), @@ -361,6 +431,13 @@ void QItem_Handle_BounceTo(OneQueItem *Item, StrBuf *Line, const char **Pos) StrBufExtract_NextToken(Item->BounceTo, Line, Pos, '|'); } +void QItem_Handle_SenderRoom(OneQueItem *Item, StrBuf *Line, const char **Pos) +{ + if (Item->SenderRoom == NULL) + Item->SenderRoom = NewStrBufPlain(NULL, StrLength(Line)); + StrBufExtract_NextToken(Item->SenderRoom, Line, Pos, '|'); +} + void QItem_Handle_Recipient(OneQueItem *Item, StrBuf *Line, const char **Pos) { if (Item->Current == NULL) @@ -378,7 +455,10 @@ void QItem_Handle_retry(OneQueItem *Item, StrBuf *Line, const char **Pos) { Item->Retry = StrBufExtractNext_int(Line, Pos, '|'); - Item->Retry *= 2; + if (Item->Retry == 0) + Item->Retry = SMTP_RETRY_INTERVAL; + else + Item->Retry *= 2; } @@ -398,7 +478,7 @@ void QItem_Handle_Attempted(OneQueItem *Item, StrBuf *Line, const char **Pos) /** * this one has to have the context for loading the message via the redirect buffer... */ -StrBuf *smtp_load_msg(OneQueItem *MyQItem, int n) +StrBuf *smtp_load_msg(OneQueItem *MyQItem, int n, char **Author, char **Address) { CitContext *CCC=CC; StrBuf *SendMsg; @@ -407,17 +487,19 @@ StrBuf *smtp_load_msg(OneQueItem *MyQItem, int n) CtdlOutputMsg(MyQItem->MessageID, MT_RFC822, HEADERS_ALL, 0, 1, NULL, - (ESC_DOT|SUPPRESS_ENV_TO) ); + (ESC_DOT|SUPPRESS_ENV_TO), + Author, + Address); SendMsg = CCC->redirect_buffer; CCC->redirect_buffer = NULL; if ((StrLength(SendMsg) > 0) && ChrPtr(SendMsg)[StrLength(SendMsg) - 1] != '\n') { - syslog(LOG_WARNING, - "SMTP client[%d]: Possible problem: message did not " - "correctly terminate. (expecting 0x10, got 0x%02x)\n", - MsgCount, //yes uncool, but best choice here... - ChrPtr(SendMsg)[StrLength(SendMsg) - 1] ); + SMTPC_syslog(LOG_WARNING, + "[%d] Possible problem: message did not " + "correctly terminate. (expecting 0x10, got 0x%02x)\n", + MsgCount, //yes uncool, but best choice here... + ChrPtr(SendMsg)[StrLength(SendMsg) - 1] ); StrBufAppendBufPlain(SendMsg, HKEY("\r\n"), 0); } return SendMsg; @@ -430,7 +512,7 @@ StrBuf *smtp_load_msg(OneQueItem *MyQItem, int n) * instructions for "5" codes (permanent fatal errors) and produce/deliver * a "bounce" message (delivery status notification). */ -void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt) +void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt, ParsedURL *Relay) { static int seq = 0; @@ -439,38 +521,49 @@ void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt) StrBuf *Msg = NULL; StrBuf *BounceMB; struct recptypes *valid; + time_t now; HashPos *It; void *vQE; long len; const char *Key; + int first_attempt = 0; int successful_bounce = 0; int num_bounces = 0; int give_up = 0; - syslog(LOG_DEBUG, "smtp_do_bounce() called\n"); + SMTPCM_syslog(LOG_DEBUG, "smtp_do_bounce() called\n"); + + if (MyQItem->SendBounceMail == 0) + return; - if ( (ev_time() - MyQItem->Submitted) > SMTP_GIVE_UP ) { - give_up = 1;/// TODO: replace time by libevq timer get + now = time (NULL); //ev_time(); + + if ( (now - MyQItem->Submitted) > SMTP_GIVE_UP ) { + give_up = 1; + } + + if (MyQItem->Retry == SMTP_RETRY_INTERVAL) { + first_attempt = 1; } /* * Now go through the instructions checking for stuff. */ + Msg = NewStrBufPlain(NULL, 1024); It = GetNewHashPos(MyQItem->MailQEntries, 0); while (GetNextHashPos(MyQItem->MailQEntries, It, &len, &Key, &vQE)) { MailQEntry *ThisItem = vQE; - if ((ThisItem->Status == 5) || /* failed now? */ - ((give_up == 1) && - (ThisItem->Status != 2))) + if ((ThisItem->Active && (ThisItem->Status == 5)) || /* failed now? */ + ((give_up == 1) && (ThisItem->Status != 2)) || + ((first_attempt == 1) && (ThisItem->Status != 2))) /* giving up after failed attempts... */ { - if (num_bounces == 0) - Msg = NewStrBufPlain(NULL, 1024); ++num_bounces; + StrBufAppendBufPlain(Msg, HKEY(" "), 0); StrBufAppendBuf(Msg, ThisItem->Recipient, 0); StrBufAppendBufPlain(Msg, HKEY(": "), 0); StrBufAppendBuf(Msg, ThisItem->StatusMessage, 0); @@ -480,13 +573,29 @@ void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt) DeleteHashPos(&It); /* Deliver the bounce if there's anything worth mentioning */ - syslog(LOG_DEBUG, "num_bounces = %d\n", num_bounces); + SMTPC_syslog(LOG_DEBUG, "num_bounces = %d\n", num_bounces); if (num_bounces == 0) { FreeStrBuf(&Msg); return; } + if ((StrLength(MyQItem->SenderRoom) == 0) && MyQItem->HaveRelay) { + const char *RelayUrlStr = "[not found]"; + /* one message that relaying is broken is enough; no extra room error message. */ + StrBuf *RelayDetails = NewStrBuf(); + + if (Relay != NULL) + RelayUrlStr = ChrPtr(Relay->URL); + + StrBufPrintf(RelayDetails, + "Relaying via %s failed permanently. \n Reason:\n%s\n Revalidate your relay configuration.", + RelayUrlStr, + ChrPtr(Msg)); + CtdlAideMessage(ChrPtr(RelayDetails), "Relaying Failed"); + FreeStrBuf(&RelayDetails); + } + boundary = NewStrBufPlain(HKEY("=_Citadel_Multipart_")); StrBufAppendPrintf(boundary, "%s_%04x%04x", @@ -502,7 +611,7 @@ void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt) StrLength(OMsgTxt)); /* the original message */ if (BounceMB == NULL) { FreeStrBuf(&boundary); - syslog(LOG_ERR, "Failed to alloc() bounce message.\n"); + SMTPCM_syslog(LOG_ERR, "Failed to alloc() bounce message.\n"); return; } @@ -511,7 +620,7 @@ void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt) if (bmsg == NULL) { FreeStrBuf(&boundary); FreeStrBuf(&BounceMB); - syslog(LOG_ERR, "Failed to alloc() bounce message.\n"); + SMTPCM_syslog(LOG_ERR, "Failed to alloc() bounce message.\n"); return; } @@ -552,8 +661,19 @@ void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt) StrBufAppendBuf(BounceMB, Msg, 0); FreeStrBuf(&Msg); + if (StrLength(MyQItem->SenderRoom) > 0) + { + StrBufAppendBufPlain( + BounceMB, + HKEY("The message was originaly posted in: "), 0); + StrBufAppendBuf(BounceMB, MyQItem->SenderRoom, 0); + StrBufAppendBufPlain( + BounceMB, + HKEY("\n"), 0); + } + /* Attach the original message */ - StrBufAppendBufPlain(BounceMB, HKEY("--"), 0); + StrBufAppendBufPlain(BounceMB, HKEY("\r\n--"), 0); StrBufAppendBuf(BounceMB, boundary, 0); StrBufAppendBufPlain(BounceMB, HKEY("\r\n"), 0); StrBufAppendBufPlain(BounceMB, @@ -581,11 +701,13 @@ void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt) bmsg->cm_fields['M'] = SmashStrBuf(&BounceMB); /* First try the user who sent the message */ - if (StrLength(MyQItem->BounceTo) == 0) - syslog(LOG_ERR, "No bounce address specified\n"); - else - syslog(LOG_DEBUG, "bounce to user? <%s>\n", + if (StrLength(MyQItem->BounceTo) == 0) { + SMTPCM_syslog(LOG_ERR, "No bounce address specified\n"); + } + else { + SMTPC_syslog(LOG_DEBUG, "bounce to user? <%s>\n", ChrPtr(MyQItem->BounceTo)); + } /* Can we deliver the bounce to the original sender? */ valid = validate_recipients(ChrPtr(MyQItem->BounceTo), NULL, 0); @@ -603,16 +725,95 @@ void smtpq_do_bounce(OneQueItem *MyQItem, StrBuf *OMsgTxt) free_recipients(valid); FreeStrBuf(&boundary); CtdlFreeMessage(bmsg); - syslog(LOG_DEBUG, "Done processing bounces\n"); + SMTPCM_syslog(LOG_DEBUG, "Done processing bounces\n"); } +ParsedURL *LoadRelayUrls(OneQueItem *MyQItem, + char *Author, + char *Address) +{ + int nRelays = 0; + ParsedURL *RelayUrls = NULL; + char mxbuf[SIZ]; + ParsedURL **Url = &MyQItem->URL; + + nRelays = get_hosts(mxbuf, "fallbackhost"); + if (nRelays > 0) { + StrBuf *All; + StrBuf *One; + const char *Pos = NULL; + All = NewStrBufPlain(mxbuf, -1); + One = NewStrBufPlain(NULL, StrLength(All) + 1); + + while ((Pos != StrBufNOTNULL) && + ((Pos == NULL) || + !IsEmptyStr(Pos))) + { + StrBufExtract_NextToken(One, All, &Pos, '|'); + if (!ParseURL(Url, One, DefaultMXPort)) { + SMTPC_syslog(LOG_DEBUG, + "Failed to parse: %s\n", + ChrPtr(One)); + } + else { + (*Url)->IsRelay = 1; + MyQItem->HaveRelay = 1; + } + } + FreeStrBuf(&All); + FreeStrBuf(&One); + } + nRelays = get_hosts(mxbuf, "smarthost"); + if (nRelays > 0) { + char *User; + StrBuf *All; + StrBuf *One; + const char *Pos = NULL; + All = NewStrBufPlain(mxbuf, -1); + One = NewStrBufPlain(NULL, StrLength(All) + 1); + + while ((Pos != StrBufNOTNULL) && + ((Pos == NULL) || + !IsEmptyStr(Pos))) + { + StrBufExtract_NextToken(One, All, &Pos, '|'); + User = strchr(ChrPtr(One), ' '); + if (User != NULL) { + if (!strcmp(User + 1, Author) || + !strcmp(User + 1, Address)) + StrBufCutAt(One, 0, User); + else { + MyQItem->HaveRelay = 1; + continue; + } + } + if (!ParseURL(Url, One, DefaultMXPort)) { + SMTPC_syslog(LOG_DEBUG, + "Failed to parse: %s\n", + ChrPtr(One)); + } + else { + ///if (!Url->IsIP)) // todo dupe me fork ipv6 + (*Url)->IsRelay = 1; + MyQItem->HaveRelay = 1; + } + } + FreeStrBuf(&All); + FreeStrBuf(&One); + } + return RelayUrls; +} /* * smtp_do_procmsg() * * Called by smtp_do_queue() to handle an individual message. */ void smtp_do_procmsg(long msgnum, void *userdata) { + time_t now; + int mynumsessions = num_sessions; struct CtdlMessage *msg = NULL; + char *Author = NULL; + char *Address = NULL; char *instr = NULL; StrBuf *PlainQItem; OneQueItem *MyQItem; @@ -621,17 +822,22 @@ void smtp_do_procmsg(long msgnum, void *userdata) { void *vQE; long len; const char *Key; - int nRelays = 0; - ParsedURL *RelayUrls = NULL; int HaveBuffers = 0; StrBuf *Msg =NULL; - syslog(LOG_DEBUG, "SMTP Queue: smtp_do_procmsg(%ld)\n", msgnum); + if (mynumsessions > max_sessions_for_outbound_smtp) { + SMTPC_syslog(LOG_DEBUG, + "skipping because of num jobs %d > %d max_sessions_for_outbound_smtp", + mynumsessions, + max_sessions_for_outbound_smtp); + } + + SMTPC_syslog(LOG_DEBUG, "smtp_do_procmsg(%ld)\n", msgnum); ///strcpy(envelope_from, ""); msg = CtdlFetchMessage(msgnum, 1); if (msg == NULL) { - syslog(LOG_ERR, "SMTP Queue: tried %ld but no such message!\n", + SMTPC_syslog(LOG_ERR, "tried %ld but no such message!\n", msgnum); return; } @@ -652,20 +858,23 @@ void smtp_do_procmsg(long msgnum, void *userdata) { FreeStrBuf(&PlainQItem); if (MyQItem == NULL) { - syslog(LOG_ERR, - "SMTP Queue: Msg No %ld: already in progress!\n", - msgnum); + SMTPC_syslog(LOG_ERR, + "Msg No %ld: already in progress!\n", + msgnum); return; /* s.b. else is already processing... */ } /* * Postpone delivery if we've already tried recently. */ + now = time(NULL); if ((MyQItem->ReattemptWhen != 0) && - (time(NULL) < MyQItem->ReattemptWhen) && + (now < MyQItem->ReattemptWhen) && (run_queue_now == 0)) { - syslog(LOG_DEBUG, "SMTP client: Retry time not yet reached.\n"); + SMTPC_syslog(LOG_DEBUG, + "Retry time not yet reached. %ld seconds left.", + MyQItem->ReattemptWhen - now); It = GetNewHashPos(MyQItem->MailQEntries, 0); pthread_mutex_lock(&ActiveQItemsLock); @@ -687,7 +896,7 @@ void smtp_do_procmsg(long msgnum, void *userdata) { * Bail out if there's no actual message associated with this */ if (MyQItem->MessageID < 0L) { - syslog(LOG_ERR, "SMTP Queue: no 'msgid' directive found!\n"); + SMTPCM_syslog(LOG_ERR, "no 'msgid' directive found!\n"); It = GetNewHashPos(MyQItem->MailQEntries, 0); pthread_mutex_lock(&ActiveQItemsLock); { @@ -704,79 +913,96 @@ void smtp_do_procmsg(long msgnum, void *userdata) { return; } - { - char mxbuf[SIZ]; - ParsedURL **Url = &MyQItem->URL; - nRelays = get_hosts(mxbuf, "smarthost"); - if (nRelays > 0) { - StrBuf *All; - StrBuf *One; - const char *Pos = NULL; - All = NewStrBufPlain(mxbuf, -1); - One = NewStrBufPlain(NULL, StrLength(All) + 1); - - while ((Pos != StrBufNOTNULL) && - ((Pos == NULL) || - !IsEmptyStr(Pos))) - { - StrBufExtract_NextToken(One, All, &Pos, '|'); - if (!ParseURL(Url, One, 25)) - syslog(LOG_DEBUG, - "Failed to parse: %s\n", - ChrPtr(One)); - else { - ///if (!Url->IsIP)) // todo dupe me fork ipv6 - Url = &(*Url)->Next; - } - } - FreeStrBuf(&All); - FreeStrBuf(&One); - } - - Url = &MyQItem->FallBackHost; - nRelays = get_hosts(mxbuf, "fallbackhost"); - if (nRelays > 0) { - StrBuf *All; - StrBuf *One; - const char *Pos = NULL; - All = NewStrBufPlain(mxbuf, -1); - One = NewStrBufPlain(NULL, StrLength(All) + 1); - - while ((Pos != StrBufNOTNULL) && - ((Pos == NULL) || - !IsEmptyStr(Pos))) - { - StrBufExtract_NextToken(One, All, &Pos, '|'); - if (!ParseURL(Url, One, 25)) - syslog(LOG_DEBUG, - "Failed to parse: %s\n", - ChrPtr(One)); - else - Url = &(*Url)->Next; - } - FreeStrBuf(&All); - FreeStrBuf(&One); - } - } It = GetNewHashPos(MyQItem->MailQEntries, 0); while (GetNextHashPos(MyQItem->MailQEntries, It, &len, &Key, &vQE)) { MailQEntry *ThisItem = vQE; - syslog(LOG_DEBUG, "SMTP Queue: Task: <%s> %d\n", - ChrPtr(ThisItem->Recipient), - ThisItem->Active); + SMTPC_syslog(LOG_DEBUG, "SMTP Queue: Task: <%s> %d\n", + ChrPtr(ThisItem->Recipient), + ThisItem->Active); } DeleteHashPos(&It); - MyQItem->ActiveDeliveries = CountActiveQueueEntries(MyQItem); + MyQItem->NotYetShutdownDeliveries = + MyQItem->ActiveDeliveries = CountActiveQueueEntries(MyQItem, 1); + + /* failsafe against overload: + * will we exceed the limit set? + */ + if ((MyQItem->ActiveDeliveries + mynumsessions > max_sessions_for_outbound_smtp) && + /* if yes, did we reach more than half of the quota? */ + ((mynumsessions * 2) > max_sessions_for_outbound_smtp) && + /* if... would we ever fit into half of the quota?? */ + (((MyQItem->ActiveDeliveries * 2) < max_sessions_for_outbound_smtp))) + { + /* abort delivery for another time. */ + SMTPC_syslog(LOG_DEBUG, + "SMTP Queue: skipping because of num jobs %d + %ld > %d max_sessions_for_outbound_smtp", + mynumsessions, + MyQItem->ActiveDeliveries, + max_sessions_for_outbound_smtp); + + FreeQueItem(&MyQItem); + + return; + } + + if (MyQItem->ActiveDeliveries > 0) { + ParsedURL *RelayUrls = NULL; + int nActivated = 0; int n = MsgCount++; int m = MyQItem->ActiveDeliveries; int i = 1; - Msg = smtp_load_msg(MyQItem, n); + It = GetNewHashPos(MyQItem->MailQEntries, 0); + + Msg = smtp_load_msg(MyQItem, n, &Author, &Address); + RelayUrls = LoadRelayUrls(MyQItem, Author, Address); + if ((RelayUrls == NULL) && MyQItem->HaveRelay) { + + while ((i <= m) && + (GetNextHashPos(MyQItem->MailQEntries, + It, &len, &Key, &vQE))) + { + int KeepBuffers = (i == m); + MailQEntry *ThisItem = vQE; + StrBufPrintf(ThisItem->StatusMessage, + "No relay configured matching %s / %s", + (Author != NULL)? Author : "", + (Address != NULL)? Address : ""); + ThisItem->Status = 5; + + nActivated++; + + if (i > 1) n = MsgCount++; + syslog(LOG_INFO, + "SMTPC: giving up on <%ld> <%s> %d / %d \n", + MyQItem->MessageID, + ChrPtr(ThisItem->Recipient), + i, + m); + (*((int*) userdata)) ++; + smtp_try_one_queue_entry(MyQItem, + ThisItem, + Msg, + KeepBuffers, + n, + RelayUrls); + + if (KeepBuffers) HaveBuffers = 1; + + i++; + } + DeleteHashPos(&It); + + return; + } + if (Author != NULL) free (Author); + if (Address != NULL) free (Address); + while ((i <= m) && (GetNextHashPos(MyQItem->MailQEntries, It, &len, &Key, &vQE))) @@ -786,13 +1012,19 @@ void smtp_do_procmsg(long msgnum, void *userdata) { if (ThisItem->Active == 1) { int KeepBuffers = (i == m); + + nActivated++; + if (nActivated % ndelay_count == 0) + usleep(delay_msec); + if (i > 1) n = MsgCount++; syslog(LOG_DEBUG, - "SMTPQ: Trying <%ld> <%s> %d / %d \n", + "SMTPC: Trying <%ld> <%s> %d / %d \n", MyQItem->MessageID, ChrPtr(ThisItem->Recipient), i, m); + (*((int*) userdata)) ++; smtp_try_one_queue_entry(MyQItem, ThisItem, Msg, @@ -824,19 +1056,18 @@ void smtp_do_procmsg(long msgnum, void *userdata) { const char* Key; void *VData; - syslog(LOG_WARNING, - "SMTP cleanup: unable to find " - "QItem with ID[%ld]", - MyQItem->MessageID); + SMTPC_syslog(LOG_WARNING, + "unable to find QItem with ID[%ld]", + MyQItem->MessageID); while (GetNextHashPos(ActiveQItems, It, &len, &Key, &VData)) { - syslog(LOG_WARNING, - "SMTP cleanup: have: ID[%ld]", - ((OneQueItem *)VData)->MessageID); + SMTPC_syslog(LOG_WARNING, + "have: ID[%ld]", + ((OneQueItem *)VData)->MessageID); } } @@ -862,18 +1093,14 @@ void smtp_do_procmsg(long msgnum, void *userdata) { * Run through the queue sending out messages. */ void smtp_do_queue(void) { - static int is_running = 0; int num_processed = 0; - - if (is_running) - return; /* Concurrency check - only one can run */ - is_running = 1; + int num_activated = 0; pthread_setspecific(MyConKey, (void *)&smtp_queue_CC); - syslog(LOG_INFO, "SMTP client: processing outbound queue"); + SMTPCM_syslog(LOG_INFO, "processing outbound queue"); if (CtdlGetRoom(&CC->room, SMTP_SPOOLOUT_ROOM) != 0) { - syslog(LOG_ERR, "Cannot find room <%s>", SMTP_SPOOLOUT_ROOM); + SMTPC_syslog(LOG_ERR, "Cannot find room <%s>", SMTP_SPOOLOUT_ROOM); } else { num_processed = CtdlForEachMessage(MSGS_ALL, @@ -882,14 +1109,12 @@ void smtp_do_queue(void) { SPOOLMIME, NULL, smtp_do_procmsg, - NULL); + &num_activated); } - syslog(LOG_INFO, - "SMTP client: queue run completed; %d messages processed", - num_processed); + SMTPC_syslog(LOG_INFO, + "queue run completed; %d messages processed %d activated", + num_processed, num_activated); - run_queue_now = 0; - is_running = 0; } @@ -904,7 +1129,7 @@ void smtp_init_spoolout(void) { * Create the room. This will silently fail if the room already * exists, and that's perfectly ok, because we want it to exist. */ - CtdlCreateRoom(SMTP_SPOOLOUT_ROOM, 3, "", 0, 1, 0, VIEW_MAILBOX); + CtdlCreateRoom(SMTP_SPOOLOUT_ROOM, 3, "", 0, 1, 0, VIEW_QUEUE); /* * Make sure it's set to be a "system room" so it doesn't show up @@ -962,28 +1187,46 @@ void cmd_smtp(char *argbuf) { CTDL_MODULE_INIT(smtp_queu) { + char *pstr; + if (!threading) { + pstr = getenv("CITSERVER_n_session_max"); + if ((pstr != NULL) && (*pstr != '\0')) + max_sessions_for_outbound_smtp = atol(pstr); /* how many sessions might be active till we stop adding more smtp jobs */ + + pstr = getenv("CITSERVER_smtp_n_delay_count"); + if ((pstr != NULL) && (*pstr != '\0')) + ndelay_count = atol(pstr); /* every n queued messages we will sleep... */ + + pstr = getenv("CITSERVER_smtp_delay"); + if ((pstr != NULL) && (*pstr != '\0')) + delay_msec = atol(pstr) * 1000; /* this many seconds. */ + + + + CtdlFillSystemContext(&smtp_queue_CC, "SMTP_Send"); ActiveQItems = NewHash(1, lFlathash); pthread_mutex_init(&ActiveQItemsLock, NULL); QItemHandlers = NewHash(0, NULL); - Put(QItemHandlers, HKEY("msgid"), QItem_Handle_MsgID, reference_free_handler); - Put(QItemHandlers, HKEY("envelope_from"), QItem_Handle_EnvelopeFrom, reference_free_handler); - Put(QItemHandlers, HKEY("retry"), QItem_Handle_retry, reference_free_handler); - Put(QItemHandlers, HKEY("attempted"), QItem_Handle_Attempted, reference_free_handler); - Put(QItemHandlers, HKEY("remote"), QItem_Handle_Recipient, reference_free_handler); - Put(QItemHandlers, HKEY("bounceto"), QItem_Handle_BounceTo, reference_free_handler); - Put(QItemHandlers, HKEY("submitted"), QItem_Handle_Submitted, reference_free_handler); + RegisterQItemHandler(HKEY("msgid"), QItem_Handle_MsgID); + RegisterQItemHandler(HKEY("envelope_from"), QItem_Handle_EnvelopeFrom); + RegisterQItemHandler(HKEY("retry"), QItem_Handle_retry); + RegisterQItemHandler(HKEY("attempted"), QItem_Handle_Attempted); + RegisterQItemHandler(HKEY("remote"), QItem_Handle_Recipient); + RegisterQItemHandler(HKEY("bounceto"), QItem_Handle_BounceTo); + RegisterQItemHandler(HKEY("source_room"), QItem_Handle_SenderRoom); + RegisterQItemHandler(HKEY("submitted"), QItem_Handle_Submitted); smtp_init_spoolout(); - CtdlRegisterCleanupHook(smtp_evq_cleanup); + CtdlRegisterEVCleanupHook(smtp_evq_cleanup); CtdlRegisterProtoHook(cmd_smtp, "SMTP", "SMTP utility commands"); - CtdlRegisterSessionHook(smtp_do_queue, EVT_TIMER); + CtdlRegisterSessionHook(smtp_do_queue, EVT_TIMER, PRIO_SEND + 10); } /* return our Subversion id for the Log */