docusign-esign-csharp-client icon indicating copy to clipboard operation
docusign-esign-csharp-client copied to clipboard

Solution is not compatible with RestSharp 107.0.1

Open rozeboosje opened this issue 3 years ago • 26 comments

For compatibility reasons we have done a rebuild of this client targeting .NET Framework 4.8

We also like to keep our NuGet packages up to date

You're using a NuGet package for RestSharp by John Sheehan, Andrew Young, Alexey Zimarev and RestSharp community

They have released a build 107.0.1 which breaks completely with the 106 builds - for example all the Interfaces are gone and they have removed synchronous methods.

At the moment this package is only used by DocuSign and not by any other 3rd party components so this is not going to cause us any difficulties, but if we ever decide to start using another 3rd party component that uses RestSharp 107.0.1 we will be in trouble.

Would it be possible to modify your code to use RestSharp 107.0.1 or even remove dependencies to this component completely?

rozeboosje avatar Jan 07 '22 15:01 rozeboosje

I attempted to upgrade to Docusign C# SDK 5.9 and everything just crashes, because of the RestSharp dependency.

I get "System.MissingMethodException: Method not found: 'Void RestSharp.RestClient.set_Proxy(System.Net.IWebProxy)'.", when I attempt to instantiate an APIClient.

var apiClient = new ApiClient();

grippstick avatar Feb 24 '22 17:02 grippstick

We had similar issues after attempting an upgrade to RestSharp v107. Please remove the dependency or upgrade asap.

joshjeppson avatar Mar 16 '22 18:03 joshjeppson

Issues with RestSharp v1.7.0 Had to revert to v1.6.15. Please upgrade as soon as possible and let us know.

warrenharding avatar Mar 16 '22 19:03 warrenharding

It's absurd to say the least that you should choose to use a third party product like "Restsharp" to do something as basic as calling Restful APIs. For Pete's sake, just remove this reference and call the APIs directly. I have done this for other REST APIs and it ain't Rocket Surgery

rozeboosje avatar Mar 16 '22 19:03 rozeboosje

Any update on this?

comicon1234 avatar Mar 29 '22 15:03 comicon1234

@dmartin00 is there anything happening on this? This has caused problems for over 2 months now.

rozeboosje avatar Mar 29 '22 17:03 rozeboosje

+1 - I'm having this exactly problem!

codsay avatar Apr 06 '22 04:04 codsay

Hellooooooo.... Helloooooooo

faint echo

Anybody home?

rozeboosje avatar Apr 06 '22 08:04 rozeboosje

knock knock! I'm crying out.

daothanhduy305 avatar Apr 06 '22 14:04 daothanhduy305

Sorry for the lack of updates here. This is Jeremy from Docusign's SDK team.

We synced internally on this matter and are evaluating the best fix (I can't share the plan just yet, but basically we were discussing updating RestSharp vs removing the dependency), based on feedback both here and in other messages directly to us regarding challenges with RestSharp (there have been a few).

In the past few days we agreed that this will be the priority issue to resolve for our next SDK update. Unfortunately, I don't yet have an ETA. I know it will not be resolved this week or next, based on our initial assessment. I'll share an update for everyone here next week on the status, however.

jglassenberg avatar Apr 06 '22 16:04 jglassenberg

@jglassenberg Thank you! I'd be happy to wait now, knowing that it will be addressed.

rozeboosje avatar Apr 06 '22 17:04 rozeboosje

I was hoping for a more clear update by mid-week, but unfortunately, our developers were caught up in another SDK issue much of this week. We're still shooting for a better update by end of week. Sorry for the continued holdup.

jglassenberg avatar Apr 14 '22 04:04 jglassenberg

Update for this week - Engineering began working on the matter this week, but I don't expect a resolution this week or next. We're still evaluating options and scope will depend on what we select. But I'll continue to post an update here weekly at minimum.

jglassenberg avatar Apr 22 '22 04:04 jglassenberg

That's great, Jeremy. Knowing that it's in hand I'd be very happy to wait for as long as it takes.

rozeboosje avatar Apr 22 '22 13:04 rozeboosje

I didn't get to touch base with the engineer working on this issue today, and they're out Friday. But a fix is in development at least. I should have a proper update early next week.

jglassenberg avatar Apr 29 '22 06:04 jglassenberg

@jglassenberg Any news on this?

Alstria avatar May 04 '22 09:05 Alstria

Development has continued but I can't share details until we have something more definitive, unfortunately. Sorry that I still don't have a great update, other than that it is an active ticket for our current sprint.

jglassenberg avatar May 06 '22 18:05 jglassenberg

Sorry for not posting an update last week. I was waiting to confirm what we can share. And basically, we have a confirmed solution for this dependency, but the estimate is for a release in June.

I'll continue to keep you all updated on this, but not weekly. I'll follow up end of month to confirm that the fix is on track. I hope that's alright with you all.

jglassenberg avatar May 17 '22 06:05 jglassenberg

Thanks Jeremy. I'm happy to wait as long as you need to get it fully tested. So far we have been able to continue moving forward with the last nuGet package that didn't have this issue, without the rest of our Solution having a conniption. As long as that remains the case there is no mad urgency.

rozeboosje avatar May 17 '22 11:05 rozeboosje

@rozeboosje That's good to hear. We should have a fix within the coming weeks, so hopefully your solution can work at least until then.

jglassenberg avatar May 20 '22 23:05 jglassenberg

End of month update, that the fix is on track. I can't provide too specific an ETA but am expecting the fix to be released in the month of June.

jglassenberg avatar May 28 '22 16:05 jglassenberg

Semi-weekly update: still on track for a June fix.

jglassenberg avatar Jun 10 '22 23:06 jglassenberg

I'm on vacation at the moment, so I haven't gotten to sync with the engineers working on this fix. I can acknowledge that a fix is under review and still planned for a release end of month.

jglassenberg avatar Jun 26 '22 05:06 jglassenberg

News of the week: we have a release candidate for the SDK with a fix, at https://www.nuget.org/packages/DocuSign.eSign.dll/6.0.0-rc. This version removes the RestSharp dependency and upgrades Newtonsoft.Json to the latest version. If any of you can give this a try and let me know how it goes while we continue to test it, much appreciated.

jglassenberg avatar Jul 05 '22 22:07 jglassenberg

@jglassenberg I've migrated our application to SDK 6.0.0-rc from something even older than an SDK version with RestSharp. And, it goes pretty well. Everything I need is working as I expected. BTW Is there a release date for the next version?

ypaulau avatar Aug 29 '22 19:08 ypaulau

Sorry for the slow reply @ypaulau. We were held up with our latest release, but there was a release just last week. This should be resolved with that.

jglassenberg avatar Sep 17 '22 07:09 jglassenberg