-
Notifications
You must be signed in to change notification settings - Fork 249
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
Add Strong-Name version #262
Comments
+1 here. We are trying to reference this project in our MSFT internal application, but it also requires strong name. |
Ok I'll try add add strong naming to an upcoming version. I haven't done much with this in the past so I need to figure it out. |
Hi, thanks for the quick feedback. I was running the internal package publishing tool and hit the following error message: WARNING: File not StrongName signed: lib\netstandard2.0\Ardalis.Specification.dll So, I think the dll needs to be both "StrongName" and "AuthentiCode" signed. |
Hi @ardalis It seems like publishing package with target framework version above netcore, strong name is not required. Could you please add multi-target to Ardalis.Specification with netstandard2.0 and net6.0? https://docs.microsoft.com/en-us/dotnet/standard/assembly/strong-named Quote: For .NET Core and .NET 5+, strong-named assemblies do not provide material benefits. The runtime never validates the strong-name signature, nor does it use the strong-name for assembly binding. Since adding multi-target shouldn't be that complicated, can we get an ETA on this? |
I've done some research into this and looked at what other popular NuGet packages have done, and it looks like having you use StrongNamer (or similar tools) would be better than adding strong names to this package. |
I think you have looked into a very old thread. According to our internal tool, both MediatR and MediatR.Contracts are strong name signed. But would you try my another suggestion first? Make Ardalis.Specification multi-targets to both netstandard2.0 and net6.0. It seems like net6.0 doesn't require the strong name check. |
Yes we can certainly do that. |
What's the last status here? Do folks still need this? Should we investigate this issue further? |
I'm closing this issue. We don't plan to add strong-naming. The tooling is a pain. |
I try to reference this package in my wpf(.net 4.8 & .net 6) project which has strong-name , but unfortunately, it throw exception says this assembly need strong-name.
The text was updated successfully, but these errors were encountered: