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

Guzzle related fixes #385

Merged
merged 1 commit into from May 23, 2023

Conversation

GrahamCampbell
Copy link
Contributor

Brief Summary of Changes

This fixes a few issues:

  1. Bump minimum Guzzle versions to avoid known security vulnerabilities.
  2. Add missing implicit dependencies on other Guzzle packages.
  3. Upgrade Guzzle promises functions to avoid the old API that was deprecated in 1.4.0 (released 2020) and removed in 2.0.0.

What does this PR address?

  • GitHub issue (Add reference - #XX)
  • Refactoring
  • New feature
  • Bug fix
  • Adds more tests

Are tests included?

  • Yes
  • No

Reviewer, please note:

Related to guzzle/guzzle#3144 and guzzle/guzzle#3145.

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I ran the full test suite before pushing the changes and all of the tests pass.

@const-cloudinary const-cloudinary merged commit e5d878c into cloudinary:master May 23, 2023
7 checks passed
@const-cloudinary
Copy link
Contributor

@GrahamCampbell thank you for contribution!

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.

None yet

2 participants