Added InternalLogger for connection attempt. Bumped to NLog 4.5

Signed-off-by: Rolf Kristensen <sweaty1@hotmail.com>
fluent · Jun 19, 2021 · f07ff4d · f07ff4d
1 parent d0e932a
commit f07ff4d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 14 deletions.
diff --git a/src/NLog.Targets.Fluentd/Fluentd.cs b/src/NLog.Targets.Fluentd/Fluentd.cs
@@ -246,6 +246,7 @@ protected void EnsureConnected()
 
         private void ConnectClient()
         {
+            NLog.Common.InternalLogger.Debug("Fluentd Connecting to {0}:{1}", this.Host, this.Port);
             this.client.Connect(this.Host, this.Port);
             this.stream = this.client.GetStream();
             this.emitter = new FluentdEmitter(this.stream);
@@ -309,7 +310,7 @@ protected override void Write(LogEventInfo logEvent)
                 }
                 record.Add("stacktrace", transcodedFrames);
             }
-            if (this.IncludeAllProperties && logEvent.Properties.Count > 0)
+            if (this.IncludeAllProperties && logEvent.HasProperties)
             {
                 foreach (var property in logEvent.Properties)
                 {
@@ -344,7 +345,7 @@ protected override void Write(LogEventInfo logEvent)
 
         private static object SerializePropertyValue(string propertyKey, object propertyValue)
         {
-            if (propertyValue == null || Convert.GetTypeCode(propertyValue) != TypeCode.Object || propertyValue is decimal)
+            if (propertyValue == null || Convert.GetTypeCode(propertyValue) != TypeCode.Object)
             {
                 return propertyValue;   // immutable
             }

diff --git a/src/NLog.Targets.Fluentd/NLog.Targets.Fluentd.csproj b/src/NLog.Targets.Fluentd/NLog.Targets.Fluentd.csproj
@@ -2,6 +2,7 @@
 
   <PropertyGroup>
     <TargetFrameworks>net35;net40;net45;netstandard2.0</TargetFrameworks>
+    <DisableImplicitFrameworkReferences Condition=" '$(TargetFramework)' != 'netstandard2.0' ">true</DisableImplicitFrameworkReferences>
 
     <Description>NLog Target that emits the log entries to a fluentd node</Description>
     <Authors>Moriyoshi Koizumi</Authors>
@@ -18,22 +19,14 @@
     <RepositoryUrl>git://github.com/fluent/NLog.Targets.Fluentd</RepositoryUrl>
   </PropertyGroup>
 
-  <PropertyGroup Condition=" '$(TargetFramework)' != 'netstandard2.0' ">
-    <DisableImplicitFrameworkReferences>true</DisableImplicitFrameworkReferences>
-  </PropertyGroup>
+  <ItemGroup>
+    <PackageReference Include="NLog" Version="4.5.0" />
+    <PackageReference Include="MsgPack.Cli" Version="0.9.2" />
+  </ItemGroup>
 
   <ItemGroup Condition=" '$(TargetFramework)' != 'netstandard2.0' ">
     <Reference Include="System" />
     <Reference Include="System.Core" />
-
-    <PackageReference Include="MsgPack.Cli" Version="0.8.0" />
-    <PackageReference Include="NLog" Version="2.1.0.0" />
   </ItemGroup>
 
-  <ItemGroup Condition=" '$(TargetFramework)' == 'netstandard2.0' ">
-    <PackageReference Include="MsgPack.Cli" Version="0.9.2" />
-    <PackageReference Include="NLog" Version="4.5.0-rc03" />
-  </ItemGroup>
-
-
 </Project>