OSSP CVS Repository

ossp - Difference in ossp-pkg/petidomo/subscribe.c versions 1.16 and 1.17
Not logged in
[Honeypot]  [Browse]  [Home]  [Login]  [Reports
[Search]  [Ticket]  [Timeline
  [History

ossp-pkg/petidomo/subscribe.c 1.16 -> 1.17

--- subscribe.c  2001/01/19 13:31:11     1.16
+++ subscribe.c  2001/01/19 14:18:43     1.17
@@ -1,6 +1,6 @@
 /*
    $Source: /v/ossp/cvs/ossp-pkg/petidomo/subscribe.c,v $
-   $Revision: 1.16 $
+   $Revision: 1.17 $
 
    Copyright (C) 2000 by CyberSolutions GmbH, Germany.
 
@@ -72,7 +72,7 @@
             listname = defaultlist;
         else
             {
-            syslog(LOG_NOTICE, "%s: subscribe-command invalid: No list specified.", MailStruct->From);
+            syslog(LOG_INFO, "%s: subscribe-command invalid: No list specified.", MailStruct->From);
             fh = vOpenMailer(envelope, originator, NULL);
             if (fh != NULL)
                 {
@@ -117,8 +117,8 @@
             {
             /* Access was unauthorized, notify the originator. */
 
-            syslog(LOG_INFO, "\"%s\" tried to subscribe \"%s\" to list \"%s\", but couldn't " \
-                   "provide the correct password.", originator, address, listname);
+            syslog(LOG_INFO, "%s: Attempt to subscribe \"%s\" to list \"%s\" rejected due to lack of " \
+                   "a correct admin password.", MailStruct->From, address, listname);
 
             fh = vOpenMailer(envelope, originator, NULL);
             if (fh != NULL)
@@ -142,8 +142,7 @@
                 CloseMailer(fh);
                 }
             else
-                syslog(LOG_ERR, "Failed to send email to \"%s\" concerning his request.",
-                       originator);
+                syslog(LOG_ERR, "Failed to send email to \"%s\" concerning his request.", originator);
 
             /* Notify the owner. */
 
@@ -180,6 +179,9 @@
 
     if (isSubscribed(listname, address, NULL, NULL, FALSE) == TRUE)
         {
+        syslog(LOG_INFO, "%s: Attempt to subscribe \"%s\" to list \"%s\" rejected because the " \
+               "address is already on the list.", MailStruct->From, address, listname);
+
         /* Notify the originator, that the address is already a
            member. */
 
@@ -210,8 +212,14 @@
         {
         /* Require confirmation. */
 
-        char* command = text_easy_sprintf("subscribe %s %s", address, listname);
-        char* cookie  = queue_command(MailStruct, command);
+        char* command;
+        char* cookie;
+
+        syslog(LOG_INFO, "%s: Attempt to subscribe \"%s\" to list \"%s\" deferred because the " \
+               "request must be acknowledged first.", MailStruct->From, address, listname);
+
+        command = text_easy_sprintf("subscribe %s %s", address, listname);
+        cookie  = queue_command(MailStruct, command);
 
         /* Send request for confirmation to the user. */
 
@@ -278,6 +286,8 @@
 
     /* Okay, add the address to the list. */
 
+    syslog(LOG_INFO, "%s: Okay; subscribing address \"%s\" to list \"%s\".", MailStruct->From, address, listname);
+
     fh = fopen(ListConfig->address_file, "a");
     if (fh == NULL)
         {

CVSTrac 2.0.1