Changeset 595 for trunk/gsdl/src/recpt/receptionist.cpp

Timestamp:

1999-09-17T09:38:17+12:00 (25 years ago)

Author:

sjboddie

Message:

added some file locking stuff for logging. Windows still needs to
be done.

File:

• trunk/gsdl/src/recpt/receptionist.cpp (modified) (7 diffs)

trunk/gsdl/src/recpt/receptionist.cpp

@@ -28 +28 @@
 /*
    $Log$
+   Revision 1.31  1999/09/16 21:38:17  sjboddie
+   added some file locking stuff for logging. Windows still needs to
+   be done.
+
    Revision 1.30  1999/09/07 04:56:58  sjboddie
    added GPL notice
@@ -172 +176 @@
 #include <assert.h>
 #include <time.h>
+#include <stdio.h>
 #include <fstream.h>
 
@@ -464 +469 @@
     converthere++;
   }
-
+  
   // set default converter if no good one has been defined
   if (!defaultconvertname.empty()) {
     cgiarginfo *ainfo = argsinfo.getarginfo ("w");
-    if ((ainfo != NULL) && (converters.get_outconverter(ainfo->argdefault) == NULL)) {
-      ainfo->defaultstatus = cgiarginfo::good;
-      ainfo->argdefault = defaultconvertname;
-    }
-  }
-  
+    if (ainfo->argdefault != "w") {
+      if ((ainfo != NULL) && (converters.get_outconverter(ainfo->argdefault) == NULL)) {
+    ainfo->defaultstatus = cgiarginfo::good;
+    ainfo->argdefault = defaultconvertname;
+      }
+    }
+  }
+
   // init the actions
   actionptrmap::iterator actionhere = actions.begin ();
@@ -617 +624 @@
 bool receptionist::log_cgi_args (cgiargsclass &args, ostream &logout) {
 
-  utf8outconvertclass text_t2utf8;
-
   // see if we want to log the cgi arguments
   if (!configinfo.logcgiargs) return true;
@@ -625 +630 @@
   host = getenv("REMOTE_HOST");
   if (host == NULL) host = getenv ("REMOTE_ADDR");
+  if (host == NULL) host = "";
   char *browser = getenv("HTTP_USER_AGENT");
+  if (browser == NULL) browser = "";
   time_t ttime = time(NULL);
 
@@ -642 +649 @@
   text_t logfile = filename_cat (configinfo.gsdlhome, "etc");
   logfile = filename_cat (logfile, "usage.txt");
-  char *lfile = logfile.getcstr();
-  
+
+  text_t logstr = host;
+  logstr += " [";
+  logstr += ttime;
+  logstr += "] (" + argstr + ") \"";
+  logstr += browser;
+  logstr += "\"\n";
+
+  return append_logstr (logfile, logstr, logout);
+}
+
+bool receptionist::append_logstr (const text_t &filename, const text_t &logstr,
+                  ostream &logout) {
+
+  utf8outconvertclass text_t2utf8;
+  char *lfile = filename.getcstr();
   ofstream log (lfile, ios::app);
   
@@ -650 +671 @@
     delete lfile;
     return false;
-  } 
-
-  log << text_t2utf8 << host << " [" << ttime 
-      << "] (" << argstr << ") \"" << browser << "\"\n";
-    
+  }
+
+  int fd = log.filedesc();
+  // lock_val is set to 0 if file is locked successfully
+  int lock_val = 1;
+  WRITELOCK_FILE (fd);
+  if (lock_val == 0) {
+    log << text_t2utf8 << logstr;
+    UNLOCK_FILE (fd);
+  } else 
+    logout << "Error: Couldn't lock file " << lfile << "\n";
+
   log.close();