serilog-contrib / serilog-sinks-grafana-loki

A Serilog sink sending log events to Grafana Loki
MIT License
201 stars 30 forks source link

Public assemblies should have a strong name #215

Closed tom-englert closed 1 year ago

tom-englert commented 1 year ago

Which version of Serilog.Sinks.Grafana.Loki are you using?

8.1

Which version of .NET are you using?

net48

Describe the bug

It's not possible to use the sink with a NetFramework project that has a strong name

To Reproduce

Try to use the sink in a NetFramework project with a strong name:

At compile time:

CSC : warning CS8002: Referenced assembly 'Serilog.Sinks.Grafana.Loki, Version=8.1.0.0, Culture=neutral, PublicKeyToken=null' does not have a strong name.

At runtime:

System.IO.FileLoadException
Could not load file or assembly 'Serilog.Sinks.Grafana.Loki, Version=8.1.0.0, Culture=neutral, PublicKeyToken=null' or one of its dependencies. A strongly-named assembly is required. (Exception from HRESULT: 0x80131044)

Expected behavior

We can compile and run NetFramework projects with a strong name, using this sink.

Log/SelfLog output or exception with stacktrace

System.IO.FileLoadException Could not load file or assembly 'Serilog.Sinks.Grafana.Loki, Version=8.1.0.0, Culture=neutral, PublicKeyToken=null' or one of its dependencies. A strongly-named assembly is required. (Exception from HRESULT: 0x80131044)

Application or code sample, which could be used to reproduce a bug

No response

Additional context

No response

I have read the documentation

mishamyte commented 1 year ago

Hi!

Implemented in v8.2.0-beta.3. Could you pls validate it?

Plan to release v8.2.0 on Monday if all would be good

mishamyte commented 1 year ago

Released as a part of v8.2.0

tom-englert commented 1 year ago

8.2.0 works as expected 👍

github-actions[bot] commented 1 year ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue.