Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Public assemblies should have a strong name #215

Closed
1 task done
tom-englert opened this issue Oct 26, 2023 · 4 comments
Closed
1 task done

Public assemblies should have a strong name #215

tom-englert opened this issue Oct 26, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@tom-englert
Copy link
Contributor

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

@tom-englert tom-englert added the bug Something isn't working label Oct 26, 2023
tom-englert added a commit to tom-englert/serilog-sinks-grafana-loki that referenced this issue Oct 26, 2023
@mishamyte mishamyte added enhancement New feature or request and removed bug Something isn't working labels Oct 28, 2023
@mishamyte
Copy link
Member

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
Copy link
Member

Released as a part of v8.2.0

@tom-englert
Copy link
Contributor Author

8.2.0 works as expected 👍

Copy link

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

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Development

Successfully merging a pull request may close this issue.

2 participants