cloudinary / CloudinaryDotNet

Cloudinary DotNet library
MIT License
102 stars 68 forks source link

Important issue - CloudinaryDotNet package is using deprecated references #339

Closed ravenet closed 10 months ago

ravenet commented 10 months ago

Hello Team,

This is an important issue as I have noticed that CloudinaryDotNet package using old reference which was deprecated by Microsoft.

image

https://www.nuget.org/packages/Microsoft.AspNetCore.Html.Abstractions https://www.nuget.org/packages/Microsoft.AspNetCore.Http.Abstractions

On top of that please remove Newton.soft json library as it is a clause with Microsoft official library in serialize the response by the custom delegate.

Thank you

wissam-khalili commented 10 months ago

Hi @ravenet ,

Thank you for your input. We will review the issue and will keep you posted.

Regards, Wissam

skalahasti-cloudinary commented 10 months ago

Hi @ravenet,

Thanks for your patience.

It looks like the warning that you see is a message generated by bot. A deprecated library is not an issue.

ROfficial Microsoft library only supports the latest versions of .NET, for legacy projects we still need Newtonsoft.json. No change for now.

Thanks, Sree

ravenet commented 10 months ago

Hi Team,

Thank you and as we move forward with the latest framework why do you have to support the legacy and newest framework with the same package?

I would suggest that you create a separate package for the .net core series or we may not use this library as it is necessary to install deprecated libraries.

Thank you

skalahasti-cloudinary commented 10 months ago

Hi @ravenet ,

As of now, we do not have any plans. I am checking with the concerned team.

Thanks Sree

wissam-khalili commented 10 months ago

Hi @ravenet ,

As it is mentioned on the below official website, .NET Framework is not "legacy" and is still under full maintenance: https://dotnet.microsoft.com/en-us/download/dotnet-framework

No changes for now.

Regards, Wissam