Skip to content
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

[chore] Remove vulnerable RestSharp dependency in test suite #67

Merged
merged 2 commits into from
Oct 1, 2024

Conversation

nwithan8
Copy link
Member

@nwithan8 nwithan8 commented Oct 1, 2024

Description

Closes https://github.com/EasyPost/easyvcr-csharp/security/dependabot/1

RestSharp is solely a test dependency, and the edge-case test is no longer valid in new versions of RestSharp.

This does not require a new version and will not interfere with downstream users.

Testing

N/A

Pull Request Type

Please select the option(s) that are relevant to this PR.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Improvement (fixing a typo, updating readme, renaming a variable name, etc)

@nwithan8 nwithan8 requested a review from a team as a code owner October 1, 2024 20:45
@nwithan8 nwithan8 changed the title [chore] Bump vulnerable RestSharp dependency in test suite [chore] Remove vulnerable RestSharp dependency in test suite Oct 1, 2024
@nwithan8 nwithan8 merged commit 85b5759 into master Oct 1, 2024
13 checks passed
@nwithan8 nwithan8 deleted the restsharp_dep branch October 1, 2024 20:59
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