Update guzzle major version from 5 to 7

[phil-davis]

core dependabot PR #39368 has been merged. That bumps to 7.4.4
PR #40154 adds a bump to 7.4.5 on 2022-06-21

This is a major version update of the "guzzle" library. It provides a framework in PHP for doing outgoing client requests. It is used in things like federated sharing, where one oC10 server needs to send requests to another oC10 federated server. We have various acceptance tests for federated sharing, so automated CI will cover the common federated sharing scenarios.

Some oC10 apps mention guzzle in their composer files. We will need to coordinate update those in sync with core.

They are:

• files_external_dropbox - Bump dependencies to support guzzle v6 files_external_dropbox#116 - release issue Release 2.0.0 for Guzzle7 files_external_dropbox#133
• files_onedrive - https://github.com/owncloud/files_onedrive/pull/125 - release issue https://github.com/owncloud/files_onedrive/issues/142
• objectstore (https://github.com/owncloud/objectstore/pull/144) - release issue https://github.com/owncloud/objectstore/issues/155
• updater Update guzzle from v6 to v7 updater#626 (this one is a "bonus" not a blocker - "updater" is not an ordinary app, so it may not need to be updated exactly in sync with oC10) -> Bump guzzlehttp/guzzle to 7.4.0 updater#629
• files_external_s3 ([full-ci] Add compatibility with Guzzle 7 files_external_s3#762) - release issue Release 2.0.0 for Guzzle7 files_external_s3#933
• files_primary_s3 ([full-ci] Add compatibility with Guzzle 7 files_primary_s3#498) - release issue Release 1.3.0 for Guzzle7 files_primary_s3#577
• Calendar ([tests-only] Add compatibility with Guzzle 7 calendar#1212) - release issue Release 2.0.0 for Guzzle7 calendar#1259
• Market Add compatibility with Guzzle 7 market#817 - release issue Release 0.7.0 for Guzzle7 market#1037
• Web [WIP] [full-ci] Add compatibility with Guzzle 7 web#5933
• graphapi https://github.com/owncloud/graphapi/pull/30 - release issue https://github.com/owncloud/graphapi/issues/32
• gallery (needs investigation, mentions guzzle in composer.json require-dev only) - seems to be working OK after a manual test 2022-06-25 - see Check that gallery is working OK after Guzzle7 core update for core 10.11 gallery#859
• richdocuments Adjust getMimeType for guzzle7 dependencies richdocuments#470 - release issue Release 3.0.0 for Guzzle7 richdocuments#472

Apps that use the client, but most likely still work:

• files_external_ftp - no direct use of guzzle. Needs manual testing.
• oco_selfservice
• Onlyoffice
• Collabora
• Encryption (HSM)

Other things considered:

• files_external_gdrive Bump guzzlehttp/guzzle from 5.3.4 to 7.4.5 files_external_gdrive#76 - this repo is not a released app, so we can ignore that

Apps to pre-release:
The following apps have had their code updated so that it detects if Guzzle5 or Guzzle7 libraries are available in core, and makes the appropriate Guzzle calls. This has been done for apps that are "backend" plugins for core, that, for example, manage "core" storage of users. It is much easier to upgrade when these apps support both Guzzle5 and Guzzle7. The new version of the app can be installed while you are running core 10.9 or 10.10, and it works fine. Then you upgrade core to 10.11, and you don't have to think about how to upgrade the app "at the same time".

We can and should do QA and release new versions of these apps ASAP (well before releasing core 10.11).

• files_primary_s3 Release 1.3.0 for Guzzle7 files_primary_s3#577
• objectstore https://github.com/owncloud/objectstore/issues/155

[JammingBen]

I updated the list above with the affected apps and PRs to make it work. Most of the pipelines won't succeed until the core PR is merged. updater is optional and can be done later as it's not a standard OC app. Note that all these apps then need a new version which requires minimum OC 10.9 (this is not included in those PRs yet).

That should be pretty much it. But we can only know for sure if we take the step and merge this upgrade. So, do we want to proceed? @phil-davis @micbar @pmaier1 any objections?

[phil-davis]

That should be pretty much it. But we can only know for sure if we take the step and merge this upgrade. So, do we want to proceed? @phil-davis @micbar @pmaier1 any objections?

No objection from me. IMO it will be good to get this stuff up-to-date.

[IljaN]

Merge after 10.9, decided in planning.

Edit: 10.10 was released, and we have merged after 10.10, for release in core 10.11

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 10 days if no further activity occurs. Thank you for your contributions.

[phil-davis]

waiting on merge of #39368 to core, and then merging the various app PRs.

[phil-davis]

Also see #40103 - a new patch release of v6 and v7 came out yesterday, giving this a kick along.

[phil-davis]

Assigned myself. The main core PR #39368 has been merged.

I will follow-up the oC10 app PRs and get all CI passing for apps. And make a release/test plan with what I think needs to be done in the coming "x" weeks to manually QA various combinations of core and apps to confirm that they work with Guzzle 7.

@jnweiger FYI