You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by da...@apache.org on 2020/12/08 10:23:32 UTC

[logging-log4net] branch master updated: Fix AdoNetAppender using npgsql

This is an automated email from the ASF dual-hosted git repository.

davydm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/logging-log4net.git


The following commit(s) were added to refs/heads/master by this push:
     new 634ffca  Fix AdoNetAppender using npgsql
     new 2b8b170  Merge pull request #71 from Flexberry/fix-npgsql-appender
634ffca is described below

commit 634ffca956045cc25a8f072ca0fd05c49228a70a
Author: Tsar Nikolay <ns...@ics.perm.ru>
AuthorDate: Mon Dec 7 19:28:08 2020 +0500

    Fix AdoNetAppender using npgsql
---
 src/log4net/Appender/AdoNetAppender.cs | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/log4net/Appender/AdoNetAppender.cs b/src/log4net/Appender/AdoNetAppender.cs
index dd9939b..2f6edfb 100644
--- a/src/log4net/Appender/AdoNetAppender.cs
+++ b/src/log4net/Appender/AdoNetAppender.cs
@@ -553,8 +553,17 @@ namespace log4net.Appender
 					{
 						dbCmd.Transaction = dbTran;
 					}
-					// prepare the command, which is significantly faster
-					dbCmd.Prepare();
+
+					try
+					{
+						// prepare the command, which is significantly faster
+						dbCmd.Prepare();
+					}
+					catch (Exception)
+					{
+						// ignore prepare exceptions as they can happen without affecting actual logging, eg on npgsql
+					}
+
 					// run for all events
 					foreach (LoggingEvent e in events)
 					{