Skip to content

Update S3AdapterFactory.php#7497

Open
k99k5 wants to merge 5 commits intohyperf:masterfrom
k99k5:patch-1
Open

Update S3AdapterFactory.php#7497
k99k5 wants to merge 5 commits intohyperf:masterfrom
k99k5:patch-1

Conversation

@k99k5
Copy link
Copy Markdown

@k99k5 k99k5 commented Aug 12, 2025

Aws\Handler\GuzzleV6\GuzzleHandler class is deprecated

Fix #7322

Aws\Handler\GuzzleV6\GuzzleHandler class is deprecated
@k99k5
Copy link
Copy Markdown
Author

k99k5 commented Aug 12, 2025

@limingxinleo 有空看下

@sy-records sy-records requested a review from Copilot August 14, 2025 03:14
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the S3AdapterFactory to handle the deprecation of Aws\Handler\GuzzleV6\GuzzleHandler by implementing backward compatibility support. The change allows the factory to use the newer Aws\Handler\Guzzle\GuzzleHandler when available while falling back to the deprecated version.

  • Removes direct import of deprecated GuzzleV6\GuzzleHandler
  • Adds runtime class existence check to choose appropriate handler
  • Maintains backward compatibility with older AWS SDK versions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

limingxinleo
limingxinleo previously approved these changes Oct 28, 2025
@k99k5
Copy link
Copy Markdown
Author

k99k5 commented Oct 30, 2025

@limingxinleo 好像还没合并?

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.

[BUG] Using the "Aws\Handler\GuzzleV6\GuzzleHandler" class is deprecated, use "Aws\Handler\Guzzle\GuzzleHandler" instead.

4 participants