FSN#41140 fac_verify laten controleren of logfile schrijfbaar is

svn path=/Website/branches/v2017.1/; revision=34540

APPL/FAC/fac_verify.inc

@@ -737,6 +737,22 @@ __GROUP = "FACILITOR software";
 testfolder(rooturl + "/cust", true); // mustbethere
 testfolder(rooturl + "/temp", true); // mustbethere
 
+if (typeof __DoLog == "function")
+checker("Logfile writable?",
+            function ()
+            {
+                var result = { result: resultcodes.ok,
+                               message: " <a href='fac_StreamFile.asp?logger=0' target='_new'>View</a> the logfile"
+                             };
+                if (!__DoLog("Fac_verify test {0}".format(toDateTimeString(new Date())), "#0ff" ))
+                {
+                    result.result = resultcodes.error;
+                    result.message += "<br>" + safe.html(__LogError);
+                }
+                return result;
+            }
+        )
+
 // TODO: Zelf onder cust zoeken naar een (enkele) folder met een Oracle.udl?
 checker("Determine customerID",
             function ()

APPL/Shared/logger.inc

@@ -257,7 +257,7 @@ function __LogCloseLine(handle, s_extra, optionalColor)
 {
     if (__Logging & 1)
     {
-        __DoLogCloseLine(handle, s_extra, optionalColor)
+        return __DoLogCloseLine(handle, s_extra, optionalColor)
     }
 }
 
@@ -268,9 +268,13 @@ function __DoLogCloseLine(handle, s_extra, optionalColor)
     try
     {
         theLogger.log_close(handle, s_extra?String(s_extra):null, optionalColor);
+        return true;
     }
     catch(e)
     {  // Waarschijnlijk een permission denied omdat twee processen tegelijk proberen te schrijven
+        __LogError = e.description;
+        debugger;
+        return false;
     }
 }
 
@@ -331,10 +335,11 @@ function __DoLog(s, optionalColor, optionalCaller, leave_open)
     }
     else
         s = Server.HTMLEncode(String(s)).replace(/\n/g, "<br>");
-    __SafeDoLog(s, optionalColor, optionalCaller||__DoLog.caller, leave_open);
+    return __SafeDoLog(s, optionalColor, optionalCaller||__DoLog.caller, leave_open);
 }
 
 // Veronderstel dat s HTML-safe is
+var __LogError = "";
 function __SafeDoLog(s, optionalColor, optionalCaller, leave_open)
 {
     if (typeof Logging_disable != "undefined")
@@ -346,7 +351,9 @@ function __SafeDoLog(s, optionalColor, optionalCaller, leave_open)
     catch(e)
     {  // Waarschijnlijk een permission denied omdat twee processen tegelijk proberen te schrijven
         //throw(e);
+        __LogError = e.description;
         debugger;
+        return false;
     }
 }
 
@@ -410,13 +417,13 @@ function __DumpCollection(pCollection, title, optionalColor)
 // Log alle Request.Form variabelen in een tabel onvoorwaardelijk
 function __DoLogForm(optionalColor)
 {
-    __SafeDoLog(__DumpCollection(Request.Form, "Request.Form", optionalColor));
+    return __SafeDoLog(__DumpCollection(Request.Form, "Request.Form", optionalColor));
 }
 
 function __LogForm()
 {
     if (!(__Logging & 1)) return; // Don't bother
-    __DoLogForm();
+    return __DoLogForm();
 }
 
 // Noot: altijd jaar vooraan, goed voor logfiles, niet bedoeld voor presentatie