Skip to content

Commit

Permalink
Logger feilmelding hvis KlientMeldingId ikke var på Guid format
Browse files Browse the repository at this point in the history
  • Loading branch information
jarleborsheim committed Feb 11, 2022
1 parent b77719f commit 9527467
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 1 deletion.
1 change: 1 addition & 0 deletions KS.Fiks.IO.Client/KS.Fiks.IO.Client.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@
<PackageReference Include="Newtonsoft.Json" Version="[11.0.1, 13.0.1]" />
<PackageReference Include="KS.Fiks.QA" Version="1.0.0" PrivateAssets="All" />
<PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0" PrivateAssets="All" />
<PackageReference Include="Serilog" Version="2.10.0" />

This comment has been minimized.

Copy link
@ErHaWi

ErHaWi Feb 11, 2022

Contributor

Dere har ikke mulighet til å støtte logging gjennom ILogger i stedet? Det blir kanskje litt krunglete å dytte hele arkitekturen over på IOC, men dere kan kanskje støtte at vi sender inn en implementasjon av ILogger når vi oppretter en instans av FiksIOClient? Da kan vi som konsumenter selv styre hvilken provider som brukes og hvilke sinks som logges til etc.

This comment has been minimized.

Copy link
@jarleborsheim

jarleborsheim Feb 13, 2022

Author Contributor

Det er en god idé 👍 Tanken her var at vi nå har introdusert kode som bør logge hvis noen ting inntreffer. Det behøver ikke være Serilog nei.

This comment has been minimized.

Copy link
@jarleborsheim

jarleborsheim Feb 13, 2022

Author Contributor

Vi ser på saken.

</ItemGroup>

</Project>
18 changes: 18 additions & 0 deletions KS.Fiks.IO.Client/Models/MottattMelding.cs
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
using System.Threading.Tasks;
using KS.Fiks.IO.Client.Asic;
using KS.Fiks.IO.Client.FileIO;
using Serilog;

namespace KS.Fiks.IO.Client.Models
{
Expand All @@ -24,6 +25,23 @@ public class MottattMelding : MottattMeldingMetadata, IMottattMelding
_streamProvider = streamProvider;
_decrypter = decrypter;
_fileWriter = fileWriter;
ExtractKlientMeldingId();
}

private void ExtractKlientMeldingId()
{
if (Headere != null && Headere.ContainsKey(headerKlientMeldingId))
{
try
{
KlientMeldingId = Guid.Parse(Headere[headerKlientMeldingId]);
}
catch (Exception e)
{
Log.Error("Kunne ikke parse KlientMeldingId funnet i header til guid. KlientMeldingId: {KlientMeldingId}", Headere[headerKlientMeldingId]);
KlientMeldingId = Guid.Empty;
}
}
}

public bool HasPayload { get; }
Expand Down
4 changes: 3 additions & 1 deletion KS.Fiks.IO.Client/Models/SendtMelding.cs
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
using System;
using System.Collections.Generic;
using KS.Fiks.IO.Send.Client.Models;
using Serilog;

namespace KS.Fiks.IO.Client.Models
{
Expand All @@ -17,7 +18,8 @@ public static SendtMelding FromSentMessageApiModel(SendtMeldingApiModel sendtMel
}
catch (Exception e)
{
klientMeldingId = null;
Log.Error("Kunne ikke parse KlientMeldingId funnet i header til guid. KlientMeldingId: {KlientMeldingId}", sendtMeldingApiModel.Headere[headerKlientMeldingId]);
klientMeldingId = Guid.Empty;
}
}

Expand Down

0 comments on commit 9527467

Please sign in to comment.