Skip to content

Targeting .net standard, updated .csproj format and restsharp dependency#37

Open
bounav wants to merge 1 commit into
DocRaptor:masterfrom
bounav:updated-restsharp-dependency
Open

Targeting .net standard, updated .csproj format and restsharp dependency#37
bounav wants to merge 1 commit into
DocRaptor:masterfrom
bounav:updated-restsharp-dependency

Conversation

@bounav

@bounav bounav commented Mar 28, 2025

Copy link
Copy Markdown
Contributor
  • Using new simplified .csproj format + nuget package references
  • Targeting .net standard 2.0
  • Updated RestSharp dependency and fixed breaking code changes

Why is this change needed?

We needed to use this library in both a project that targeted .net framework and dotnetcore.
The dependency to [restsharp had a CVE.

How does it address the issue?

The dependency is updated.

Any links to any relevant tickets, articles, or other resources?

GHSA-4rr6-2v9v-wcpc

Any screenshots?

N.A.

Did you complete all of the following?

  • Run test suite? No
  • Add new tests? No
  • Consider security implications and practices? No

- Using new simplified .csproj format + nuget package references
- Targeting .net standard 2.0
- Updated RestSharp dependency and fixed breaking code changes
@bounav

bounav commented Mar 28, 2025

Copy link
Copy Markdown
Contributor Author

MB: Most of the code was generated with swagger and I didn't bother re-running the swagger stuff from scratch. I just fixed the issues and modernised the .csproj format.

@bounav

bounav commented Mar 28, 2025

Copy link
Copy Markdown
Contributor Author

This PR is linked to #28

@aespitia

Copy link
Copy Markdown

would love to see this get merged in as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants