Merge pull request #36 from Ellerbach/mtirion/fix-null-length

Small logging changes.
Ellerbach · Oct 24, 2023 · 0cc5252 · 0cc5252
2 parents 9ce3508 + 8a7ee4c
commit 0cc5252
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/src/DocLinkChecker/DocLinkChecker/Program.cs b/src/DocLinkChecker/DocLinkChecker/Program.cs
@@ -135,7 +135,7 @@ private static void DetermineConfiguration(string[] args)
             {
                 if (File.Exists(AppConstants.AppConfigFileName))
                 {
-                    console.Error($"***ERROR: {AppConstants.AppConfigFileName} already exists in this folder. We don't overwrite.");
+                    console.Error($"*** ERROR: {AppConstants.AppConfigFileName} already exists in this folder. We don't overwrite.");
 
                     // indicate we're done with an error
                     ReturnValue = ReturnValue.CommandError;
@@ -196,7 +196,7 @@ private static void ProcessCommandlineParameters(CommandLineOptions o)
                 _appConfig.ConfigFilePath = Path.GetFullPath(o.ConfigFilePath);
                 if (!File.Exists(o.ConfigFilePath))
                 {
-                    console.Error($"***ERROR: configuration file {o.ConfigFilePath} not found.");
+                    console.Error($"*** ERROR: configuration file {o.ConfigFilePath} not found.");
 
                     // indicate we're done with errors in the configuration file
                     ReturnValue = ReturnValue.ConfigurationFileErrors;
@@ -218,7 +218,7 @@ private static void ProcessCommandlineParameters(CommandLineOptions o)
                 }
                 catch (Exception ex)
                 {
-                    console.Error($"***ERROR: reading {o.ConfigFilePath} - {ex.Message}");
+                    console.Error($"*** ERROR: reading {o.ConfigFilePath} - {ex.Message}");
 
                     // indicate we're done with errors in the configuration file
                     ReturnValue = ReturnValue.ConfigurationFileErrors;
@@ -234,7 +234,7 @@ private static void ProcessCommandlineParameters(CommandLineOptions o)
 
             if (string.IsNullOrEmpty(_appConfig.DocumentationFiles.SourceFolder))
             {
-                console.Error("***ERROR: No document folder was configured, either through -d or in the config file.");
+                console.Error("*** ERROR: No document folder was configured, either through -d or in the config file.");
                 ReturnValue = ReturnValue.ConfigurationFileErrors;
                 return;
             }

diff --git a/src/DocLinkChecker/DocLinkChecker/Services/CrawlerService.cs b/src/DocLinkChecker/DocLinkChecker/Services/CrawlerService.cs
@@ -77,7 +77,7 @@ public CrawlerService(
                 }
                 catch (Exception ex)
                 {
-                    _console.Error($"*** ERROR: {ex.Message}");
+                    _console.Verbose($"*** ERROR in {nameof(ParseMarkdownFiles)} processing {file} [ignored]: {ex.Message}.\n{ex}");
                 }
             }
 

diff --git a/src/DocLinkChecker/DocLinkChecker/Services/LinkValidatorService.cs b/src/DocLinkChecker/DocLinkChecker/Services/LinkValidatorService.cs
@@ -7,6 +7,7 @@
     using System.IO;
     using System.Linq;
     using System.Net;
+    using System.Text.RegularExpressions;
     using System.Threading;
     using System.Threading.Tasks;
     using DocLinkChecker.Enums;

diff --git a/src/DocLinkChecker/DocLinkChecker/Services/ResourceValidatorService.cs b/src/DocLinkChecker/DocLinkChecker/Services/ResourceValidatorService.cs
@@ -80,7 +80,7 @@ public ResourceValidatorService(
                 }
                 catch (Exception ex)
                 {
-                    _console.Error($"*** ERROR: {ex.Message}");
+                    _console.Verbose($"*** ERROR in {nameof(CheckForOrphanedResources)} processing {resource} [ignored]: {ex.Message}.\n{ex}");
                 }
             }