AppDomainWrapper - Unhandled exception on Linux without ICU package (#…

…5188)
NLog · Apr 9, 2023 · b171b45 · b171b45
1 parent a649266
commit b171b45
Showing 1 changed file with 36 additions and 1 deletion.
diff --git a/src/NLog/Internal/AppDomains/AppDomainWrapper.cs b/src/NLog/Internal/AppDomains/AppDomainWrapper.cs
@@ -65,6 +65,16 @@ public AppDomainWrapper(AppDomain appDomain)
                 BaseDirectory = string.Empty;
             }
 
+            try
+            {
+                FriendlyName = appDomain.FriendlyName;
+            }
+            catch (Exception ex)
+            {
+                InternalLogger.Warn(ex, "AppDomain.FriendlyName Failed");
+                FriendlyName = GetFriendlyNameFromEntryAssembly() ?? GetFriendlyNameFromProcessName() ?? string.Empty;
+            }
+
             try
             {
                 ConfigurationFile = LookupConfigurationFile(appDomain);
@@ -85,7 +95,6 @@ public AppDomainWrapper(AppDomain appDomain)
                 PrivateBinPath = ArrayHelper.Empty<string>();
             }
 
-            FriendlyName = appDomain.FriendlyName;
             Id = appDomain.Id;
         }
 
@@ -115,6 +124,32 @@ private static string[] LookupPrivateBinPath(AppDomain appDomain)
 #endif
         }
 
+        private static string GetFriendlyNameFromEntryAssembly()
+        {
+            try
+            {
+                string assemblyName = Assembly.GetEntryAssembly()?.GetName()?.Name;
+                return string.IsNullOrEmpty(assemblyName) ? null : assemblyName;
+            }
+            catch
+            {
+                return null;
+            }
+        }
+
+        private static string GetFriendlyNameFromProcessName()
+        {
+            try
+            {
+                string processName = System.Diagnostics.Process.GetCurrentProcess().ProcessName;
+                return string.IsNullOrEmpty(processName) ? null : processName;
+            }
+            catch
+            {
+                return null;
+            }
+        }
+
         /// <summary>
         /// Creates an AppDomainWrapper for the current <see cref="AppDomain"/>
         /// </summary>