-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
RestSharp nuget Assembly version (105.2.3.0) vs RestSharpSigned Assembly version (100.0.0.0) #925
Comments
My understanding is that the best practice that seems to be followed by most projects now is to pin the version number of the signed assembly and use Nuget versions to track the "real" version number. Here is some discussion about it: http://james.newtonking.com/archive/2012/04/04/json-net-strong-naming-and-assembly-version-numbers I hate signing and would love to see it disappear entirely. However, the reality is that some applications need it and it is an utter cluster to do some with and some without. RestSharp especially, since it has two nuget packages (one signed, one not) and people have dependencies that have dependencies on both of them separately. This was my issue. Easypost referenced RestSharpSigned, Twilio referenced RestSharp. If you are in this situation, i've outlined your options here: EasyPost/easypost-csharp#50 (comment) I personally chose to use the Brutal Developer Strong Name Signer to change all unsigned references to signed references. Here are the steps to follow to do that |
The develop branch only contains unsigned version. I don't really want to sign it. The download numbers show that unsigned version is much wider used. |
It is sad that it was signed :( I am thinking about maintaining unsigned version |
What is the issue with using the signed version? Both signed and unsigned code can use signed assemblies. Am I missing something? |
Once you sign an assembly you cannot use it in binding redirects with previous unsigned versions. We hit pretty nasty problem with this. One of the nuget packages which we had to update also updated its reference to signed RestSharp. This automatically broke all other packages which were still with unsigned version. Basically we are stuck. In other words, until all projects which we use update their RestSharp reference to the signed one we cannot update anything OR we just build custom version of restsharp and custom version of the other project and the other project. To be honest it was a disaster for us. |
You could have reached me when you got to this problem. There are conflicting requirements from different people. Someone said - just sign it, it makes no harm. |
I think you are right. For now I think that the snk key MUST be public so that anyone who needs to do a custom build could sign the assemblies. One reference which is interesting point of view => https://www.pedrolamas.com/2016/03/01/still-strong-naming-your-assemblies-you-do-know-its-2016-right/ |
I would love to stop this signing madness. May be from like 107 I will just say that 106 was the last signed version. |
I do never sign my packages but this is me. I hope this is not just another |
Yeah, I am aware of that. I will consider to stop signing in the next major version. I can always refer to this document and say "Microsoft told me so" :) |
First of all, sorry in advance as I know the signed version is not a priority.
Hi - When I install RestSharpSigned via nuget the project reference is added as:
When installing RestSharp project reference looks like:
Is it intentional to have the version 100.0.0.0 on the signed dll or a mistake?
I know I can get round it using assembly bindings, but thought I would ask as I have been getting errors looking for version 100.0.0 file and took me a while to realise the only place that's mentioned is in the dll itself and the project references, everything else refers to 105.2.3
I've attached the what you see in the headers of the 2 dlls when decompiling them.
RestSharp.dll (unsigned)
RestSharp.dll (signed)
Thanks for any info (and also thanks for maintaining a signed version, really helps)
John
The text was updated successfully, but these errors were encountered: