FILE* is now fd

Make listcontrol get the mailname for mlmmj-bounce

3 things:

 · Check that bouncefile is not a symlink (TODO)
 · When everything went ok, return EXIT_SUCCES not EXIT_FAILURE
 · If there's a bouncemail, save it as bfilename.lastmsg

And in more readable form:

--- mlmmj-bounce.c 26 May 2004 23:25:57 -0000 1.7
+++ mlmmj-bounce.c 3 Jun 2004 06:30:39 -0000
@@ -37,16 +37,16 @@

 int main(int argc, char **argv)
 {
- int opt;
+ int opt, fd;
  char *listdir = NULL, *address = NULL, *number = NULL;
- char *bfilename, *a, *buf;
+ char *mailname = NULL, *savename, *bfilename, *a, *buf;
  size_t len;
- int fd;
  time_t t;
+ struct stat st;

  log_set_name(argv[0]);

- while ((opt = getopt(argc, argv, "hVL:a:n:")) != -1) {
+ while ((opt = getopt(argc, argv, "hVL:a:n:m:")) != -1) {
  switch(opt) {
  case 'L':
  listdir = optarg;
@@ -54,6 +54,9 @@
  case 'a':
  address = optarg;
  break;
+ case 'm':
+ mailname = optarg;
+ break;
  case 'n':
  number = optarg;
  break;
@@ -109,7 +112,14 @@
  exit(EXIT_SUCCESS); /* Not subbed, so exit silently */
  }

- /* TODO make sure the file we open below is not a symlink */
+ if(lstat(bfilename, &st) == 0) {
+ if((st.st_mode & S_IFLNK) == S_IFLNK) {
+ log_error(LOG_ARGS, "%s is a symbolic link",
+ bfilename);
+ exit(EXIT_FAILURE);
+ }
+ }
+
  if ((fd = open(bfilename, O_WRONLY|O_APPEND|O_CREAT,
  S_IRUSR|S_IWUSR)) < 0) {
  log_error(LOG_ARGS, "Could not open '%s'", bfilename);
@@ -136,5 +146,11 @@
  writen(fd, buf, strlen(buf));
  close(fd);

- return EXIT_FAILURE;
+ if(mailname) {
+ savename = concatstr(2, bfilename, ".lastmsg");
+ rename(mailname, savename);
+ free(savename);
+ }
+
+ return EXIT_SUCCESS;
 }

mlmmj-recieve deFILEified

dumpfd2fd

"Resolving" the conflict by committing my version anyway (it has a nice comment). That guy is fast.

Simply close socket if error in transmission

mygetline EINTR handling and errno = 0

fixed fgets() loop

--- src/mlmmj-send.c 2 Jun 2004 19:10:11 -0000 1.31
+++ src/mlmmj-send.c 2 Jun 2004 19:19:48 -0000
@@ -35,8 +35,6 @@
 #include "log_error.h"
 #include "mygetline.h"

-static int conncount = 0;  /* Connection count */
-
 char *bounce_from_adr(const char *recipient, const char *listadr,
        const char *mailfilename)
 {
@@ -345,15 +343,6 @@
  return 0;
 }

-void sig_child(int sig)
-{
- pid_t pid;
- int stat;
-
- while((pid = waitpid(-1, &stat, WNOHANG) > 0))
- conncount--;
-}
-
 static void print_help(const char *prg)
 {
         printf("Usage: %s [-L /path/to/list || -l listctrl] -m /path/to/mail "
@@ -380,7 +369,7 @@
 {
  size_t len = 0;
  int sockfd = 0, opt, mindex;
- int deletewhensent = 1, *newsockfd, sendres, archive = 1;
+ int deletewhensent = 1, sendres, archive = 1;
  char *listaddr, *mailfilename = NULL, *subfilename = NULL;
  char *replyto = NULL, *bounceaddr = NULL, *to_addr = NULL;
  char *relayhost = NULL, *archivefilename = NULL, *tmpstr;
@@ -389,8 +378,6 @@
  DIR *subddir;
  FILE *subfile = NULL, *mailfile = NULL, *tmpfile;
  struct dirent *dp;
- pid_t childpid;
- struct sigaction sigact;

  log_set_name(argv[0]);

@@ -564,11 +551,6 @@
  }
  free(subddirname);

- sigact.sa_handler = sig_child;
- sigemptyset(&sigact.sa_mask);
- sigact.sa_flags = SA_NOCLDSTOP;
- sigaction(SIGCHLD, &sigact, 0);
-
  while((dp = readdir(subddir)) != NULL) {
  if(!strcmp(dp->d_name, "."))
  continue;
@@ -585,41 +567,19 @@
  fprintf(stderr, "found subfile '%s'\n", subfilename);
  free(subfilename);

- while((conncount >= MAX_CONNECTIONS))
- usleep(WAITSLEEP);
-
- childpid = fork();
- if(childpid < 0)
- log_error(LOG_ARGS, "Could not fork.");
- /* TODO: we have to keep track of unsent
-  * files */
-
- conncount++;
-
- if(childpid == 0) {
- newsockfd = malloc(sizeof(int));
- initsmtp(newsockfd, relayhost);
- send_mail_many(*newsockfd, NULL, NULL,
-        mailfile, subfile, listaddr,
-        archivefilename, listdir,
-        mlmmjbounce);
- endsmtp(newsockfd);
- free(newsockfd);
- fclose(subfile);
- exit(EXIT_SUCCESS);
- } else
- fclose(subfile);
+ initsmtp(&sockfd, relayhost);
+ send_mail_many(sockfd, NULL, NULL, mailfile, subfile,
+ listaddr, archivefilename, listdir,
+ mlmmjbounce);
+ endsmtp(&sockfd);
+ fclose(subfile);
  }
  closedir(subddir);
  break;
  }

- while(conncount > 0)
- usleep(WAITSLEEP);
-
  if(archive) {
  rename(mailfilename, archivefilename);
-
  free(archivefilename);
  } else if(deletewhensent)
  unlink(mailfilename);

No need for several connections tuning for mlmmj-send

Changelog update

ChangeLog update

mlmmj-maintd now resends the ones that failed. Add listctrl switch to
do aid with it in mlmmj-send.

usleep(WAITSLEEP) while waiting for child processes to die

Fixed interrupted fgets() in write_mailbody_from_file()

Set MAX_CONNECTIONS to 1

Also remove leftover .{mailfrom,reciptto,reply-to} files without the
corresponding mail.

Fix daemonizing

mlmmj-maintd testet, and works

mlmmj-maintd work (still in progress)

append to subscribers file and not archive before last mlmmj-send is done.

Fixed leak of subfile in mlmmj-send

Better do it like this. Not pretty thought, but I doubt it can be...

Further work on mlmmj-maintd.

NEEDS TESTING!

get Cc: header as well

File for telling people the emailaddress have to be in To: or Cc:

Make sure it compiles before releasing it :-)

Move footer and customheader to control/. I've been meaning to do this
for a long time, that's where they belong.

Update for 0.5.2

it's +bounces-help not -bounces+help

Further stuff in mlmmj-maintd

Bad wording rectified

Use listdir/moderation for the mails up for moderation and use
moderation/queue as temporary queue for the mails to the moderators.

Send the From: info to listcontrol.

Adapt to the fact that we now get info from all headers matching token.

Remove basename use, since mybasename does not modify variable.

Create queue/discard directory

Update

Use the nice new mydirname and fix a bug with the archive variable.

Small bugfix. Don't allocate sizeof(char **) * count but
sizeof(char *) * count when allocating room for a new entry in a char **

It's not -L 1 but -l 1 for sending single mails

Listcontrol shouldn't open the mailfile to retrieve info we've already
retrieved in mlmmj-process. Instead we parse it on to listcontrol from
mlmmj-process.

When retrieving header info, make sure to get all headers since we might have
e.g. several To: headers.

Make this change to mailhdr, since we can have e.g. several To:
headers.

 struct mailhdr {
        const char *token;
-       char *value;
+       int valuecount;
+       char **values;
 };

added comment about the fix i committed along with the mydirname()/mybasename() patch

added mydirname() and mybasename() which doesn't mess with their argument, and
returns newly allocated strings

Add -a switch for archive [yes|no]

Use strncpy instead of memcpy
Add some free()'s
Remove some cruft

Update to reflect recent changes

Header file changes

Make mlmmj-send capable of handling a bounce from the relayhost and invoke
mlmmj-bounce to handle it like any other bounce.

This is (hopefully) the last SMTP functionality we have to implement.

-#ifdef MLMMJ_DEBUG
+#if 0

Make checkwait_smtpreply return the string that it gets from the mailserver
in case of error. That way we can check if the relayhost is bouncing local
addresses before accepting it.

add: int is_subbed(const char *listdir, const char *address) for use
in mlmmj-bounce

Handle sub/unsub confirm mail bounces, and use subscribers.d/ not
subscribers.

Mr. Boole is easy to get the wrong way. Fix so it actually works.

Dump needed data when failing in mlmmj-send.

Add mlmmj-maintd, the mlmmj maintenance daemon

mlmmj-maintd header file. Note the tunables are in mlmmj.h

Add entry about mlmmj-maintd

Also dump MAIL FROM:, RCPT TO: and Reply To: variables in the case of an
error.

Make it known by the filename that's it's a moderaterequest. That way it's
a lot easier for the maintenance process to know how to react.

Clean up and extend help everywhere.
Remove unused code from mlmmj-bounce.

Makefile goes into .cvsignore and update ChangeLog

small fix for over-sized hostnames in hostnamestr()

Send our hostname in HELO and change the mlmmj-send output from

 Connected: ok
 HELO: ok
 MAIL FROM: ok
 RCPT TO: ok
 DATA: ok 35x
 Mail queued: ok
 Closed connection

To:

 220 panther.mmj.dk ESMTP
 250 panther.mmj.dk
 250 Ok
 250 Ok
 354 End data with <CR><LF>.<CR><LF>
 250 Ok: queued as A8DA4F2FA4
 221 Bye

char *hostnamestr(void);

ChangeLog and VERSION update

Create and mention listdir/requeue

Add requeueing

Send listdir along

ChangeLog has a requeuing mechanism entry and TODO has gotten smaller :)

0.5.1 README

.cvsignore updates

explained closedlist

+0.5.1
+ o Add a file TUNABLES which documents the ways to tune mlmmj lists.
+ o Add support for Subject: prefix

if(infile == NULL) return NULL;

File for generating bounce probes.

A better TODO

added comment about why the rename() call is safe

Added WAITTIME (time to sleep before checking if we should spawn another mlmmj-send process) option in mlmmj.h

added comment about my fix

bail out if we can not open subscribers.d/

mlmmj-{,un}sub have been brought up to speed in a coordinated effort

removal of crufty crappy Makefile.in

fixed spawn count

mlmmj-make-ml.sh now creates subscribers.d as well

mlmmj-send now forks multiple connections per subscriber.d/file

added check for slash in list control string

added comment about listcontrol() cleanup

changed to be a big switch instead of a lot of if-else-if

removed (unused) BINDIR
added MY_ASSERT macro

changed incindexfile() - removed incflag, changed error return to be 0, and it now saves last used index in the file

Assorted fixes

fix from Christoph Thiel

Bouncehandling, and massive cleanups around.

Bad bad mmj. Always make sure it compiles before checkin.

This is new function to quickly stat a control of the list. Example:

int moderatedlist = statctrl(listdir, "moderated);

Then it'll return true/false according to whether a file
listdir/control/moderated exists.

Add statctrl call to listcontrol for closedlist. The other functions I've
touched are only cleaning a bit.

This is new function to quickly stat a control of the list. Example:

int moderatedlist = statctrl(listdir, "moderated);

Then it'll return true/false according to whether a file
listdir/control/moderated exists.

Add the new directories to the README. It's listdir/control, listdir/moderation
and listdir/moderation/queue

added -F (no fork) option

prepare 0.3.4 release

added comment about dot-fix

fixed smtp handling of lines which start with a dot