Merge pull request #63 from difi/common_logging

Logger req/resp i eget namespace
difi · May 18, 2016 · 05adba2 · 05adba2
2 parents c707ca6 + 883c5e3
commit 05adba2
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 30 deletions.
diff --git a/Difi.Oppslagstjeneste.Klient.Testklient/App.config b/Difi.Oppslagstjeneste.Klient.Testklient/App.config
@@ -14,34 +14,23 @@
       <factoryAdapter type="Common.Logging.Log4Net.Log4NetLoggerFactoryAdapter, Common.Logging.Log4net1213">
         <arg key="configType" value="INLINE" />
       </factoryAdapter>
-      <!--<factoryAdapter type="Common.Logging.Simple.ConsoleOutLoggerFactoryAdapter, Common.Logging">
-        <arg key="level" value="ALL" />
-        <arg key="showLogName" value="false" />
-        <arg key="showDataTime" value="false" />
-        <arg key="dateTimeFormat" value="yyyy/MM/dd HH:mm:ss:fff" />
-      </factoryAdapter>-->
     </logging>
   </common>
-
   <log4net>
     <logger name="Difi.Oppslagstjeneste.Klient">
       <appender-ref ref="RollingFileAppender" />
       <level value="DEBUG" />
     </logger>
-    <logger name="Difi.Oppslagstjeneste.RequestLog">
+    <logger additivity="false" name="Difi.Oppslagstjeneste.Klient.RequestResponse">
       <appender-ref ref="RequestRollingAppender" />
       <level value="DEBUG" />
     </logger>
     <appender name="RollingFileAppender" type="log4net.Appender.RollingFileAppender">
-      <lockingModel type="log4net.Appender.FileAppender+MinimalLock" />
+      <lockingModel type="log4net.Appender.FileAppender+MinimalLock" />  
       <file value="${AppData}\Difi\Log\" />
       <appendToFile value="true" />
-      <rollingStyle value="Date" />
       <staticLogFileName value="false" />
-      <rollingStyle value="Composite" />
-      <param name="maxSizeRollBackups" value="10" />
       <datePattern value="yyyy.MM.dd' Difi.Oppslagstjeneste-klient-dotnet.log'" />
-      <maximumFileSize value="100MB" />
       <layout type="log4net.Layout.PatternLayout">
         <conversionPattern value="%date [%thread] %-5lev - %message%newline" />
       </layout>
@@ -50,12 +39,8 @@
       <lockingModel type="log4net.Appender.FileAppender+MinimalLock" />
       <file value="${AppData}\Difi\RequestLog\" />
       <appendToFile value="true" />
-      <rollingStyle value="Date" />
       <staticLogFileName value="false" />
-      <rollingStyle value="Composite" />
-      <param name="maxSizeRollBackups" value="10" />
       <datePattern value="yyyy.MM.dd' Difi.Oppslagstjeneste-klient-dotnet.log'" />
-      <maximumFileSize value="100MB" />
       <layout type="log4net.Layout.PatternLayout">
         <conversionPattern value="%date [%thread] %-5lev - %message%newline" />
       </layout>

diff --git a/Difi.Oppslagstjeneste.Klient.Testklient/Program.cs b/Difi.Oppslagstjeneste.Klient.Testklient/Program.cs
@@ -15,7 +15,7 @@ internal class Program
         private static void Main(string[] args)
         {
             var avsendersertifikatThumbprint = CertificateIssuedToPostenNorgeAsIssuedByBuypassClass3Test4Ca3();
-            var konfigurasjon = new OppslagstjenesteKonfigurasjon(Miljø.FunksjoneltTestmiljøVerifikasjon2, avsendersertifikatThumbprint) {LoggForespørselOgRespons = false};
+            var konfigurasjon = new OppslagstjenesteKonfigurasjon(Miljø.FunksjoneltTestmiljøVerifikasjon2, avsendersertifikatThumbprint) {LoggForespørselOgRespons = true};
             Log.Debug("> Starter program!");
 
             //konfigurasjon.SendPåVegneAv = "984661185";

diff --git a/Difi.Oppslagstjeneste.Klient/OppslagstjenesteKlient.cs b/Difi.Oppslagstjeneste.Klient/OppslagstjenesteKlient.cs
@@ -20,6 +20,7 @@ namespace Difi.Oppslagstjeneste.Klient
     public class OppslagstjenesteKlient
     {
         private static readonly ILog Log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
+        private static readonly ILog RequestAndResponseLog = LogManager.GetLogger($"{typeof(OppslagstjenesteKlient).Namespace}.RequestResponse");
         private readonly OppslagstjenesteHelper _oppslagstjenesteHelper;
 
         /// <summary>
@@ -82,14 +83,14 @@ public async Task<EndringerSvar> HentEndringerAsynkront(long fraEndringsNummer,
             var requestEnvelope = new EndringerEnvelope(OppslagstjenesteKonfigurasjon.Avsendersertifikat, OppslagstjenesteKonfigurasjon.SendPåVegneAv, fraEndringsNummer, informasjonsbehov);
 
             Log.Debug($"HentEndringerAsynkront(fraEndringsNummer:{fraEndringsNummer} , informasjonsbehov:{informasjonsbehov})");
-            if (Log.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
+            if (RequestAndResponseLog.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
             {
-                Log.Debug(requestEnvelope.XmlDocument.OuterXml);
+                RequestAndResponseLog.Debug(requestEnvelope.XmlDocument.OuterXml);
             }
             var responseDocument = await GetClient().SendAsync(requestEnvelope);
-            if (Log.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
+            if (RequestAndResponseLog.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
             {
-                Log.Debug(responseDocument.Envelope.InnerXml);
+                RequestAndResponseLog.Debug(responseDocument.Envelope.InnerXml);
             }
             var dtoObject = ValidateAndConvertToDtoObject<HentEndringerRespons>(requestEnvelope, responseDocument);
             return DtoConverter.ToDomainObject(dtoObject);
@@ -134,15 +135,15 @@ public async Task<IEnumerable<Person>> HentPersonerAsynkront(string[] personiden
         {
             var requestEnvelope = new PersonsEnvelope(OppslagstjenesteKonfigurasjon.Avsendersertifikat, OppslagstjenesteKonfigurasjon.SendPåVegneAv, personidentifikator, informasjonsbehov);
             Log.Debug($"HentPersonerAsynkront(personidentifikator:{personidentifikator} , informasjonsbehov:{informasjonsbehov})");
-            if (Log.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
+            if (RequestAndResponseLog.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
             {
-                Log.Debug(requestEnvelope.XmlDocument.OuterXml);
+                RequestAndResponseLog.Debug(requestEnvelope.XmlDocument.OuterXml);
             }
             var responseDocument = await GetClient().SendAsync(requestEnvelope);
 
-            if (Log.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
+            if (RequestAndResponseLog.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
             {
-                Log.Debug(responseDocument.Envelope.InnerXml);
+                RequestAndResponseLog.Debug(responseDocument.Envelope.InnerXml);
             }
             var dtoObject = ValidateAndConvertToDtoObject<HentPersonerRespons>(requestEnvelope, responseDocument);
             var domainObject = DtoConverter.ToDomainObject(dtoObject);
@@ -174,14 +175,14 @@ public async Task<PrintSertifikatSvar> HentPrintSertifikatAsynkront()
         {
             var requestEnvelope = new PrintCertificateEnvelope(OppslagstjenesteKonfigurasjon.Avsendersertifikat, OppslagstjenesteKonfigurasjon.SendPåVegneAv);
             Log.Debug("HentPrintSertifikatAsynkront");
-            if (Log.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
+            if (RequestAndResponseLog.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
             {
-                Log.Debug(requestEnvelope.XmlDocument.OuterXml);
+                RequestAndResponseLog.Debug(requestEnvelope.XmlDocument.OuterXml);
             }
             var responseDocument = await GetClient().SendAsync(requestEnvelope);
-            if (Log.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
+            if (RequestAndResponseLog.IsDebugEnabled && OppslagstjenesteKonfigurasjon.LoggForespørselOgRespons)
             {
-                Log.Debug(responseDocument.Envelope.InnerXml);
+                RequestAndResponseLog.Debug(responseDocument.Envelope.InnerXml);
             }
             var dtoObject = ValidateAndConvertToDtoObject<HentPrintSertifikatRespons>(requestEnvelope, responseDocument);
             return DtoConverter.ToDomainObject(dtoObject);