suddenly require zip installation

meyer59 commented 6 months ago

Bug report for Cloudinary PHP Transformation Builder SDK

Describe the bug in a sentence or two.

Since the last update, this package installation is broken on our pieline. The cause is that this package require a zip installation. Is this behavior normal? For what i see from the composer.lock, the url of the package has changed from this "url": "https://github.com/cloudinary/php-transformation-builder-sdk", to this: "url": "git@github.com:cloudinary/php-transformation-builder-sdk.git", See the screenshot below for more details. As a customer, we have been using this package for years with no issues. There migth be something wrong with your packagist

Issue Type (Can be multiple)

[ x] Build - Can’t install or import the SDK
[ ] Performance - Performance issues
[ ] Behaviour - Functions aren’t working as expected (such as generate URL)
[ ] Documentation - Inconsistency between the docs and behaviour
[ ] Other (Specify)

Steps to reproduce

Composer update to the latest version (1.4)

Error screenshots or Stack Trace (if applicable)

Operating System

[ ] Linux
[ x] Windows
[ ] macOS
[ ] All

Environment and Frameworks (fill in the version numbers)

PHP Version - 8.1
Framework - Laravel 10.47

dannyv-cloudinary commented 6 months ago

Hey @meyer59 .

Could you please explain a little more about how this breaks your pipeline? Are there any specific errors or console output that you can provide?

Thanks, -Danny

meyer59 commented 5 months ago

Hey It requires the Zip php extension to be enabled otherwise it skip the package installation. Rolling back the version does solved the issue though

tamaracloudinary commented 5 months ago

Hi there @meyer59, there were no changes related to composer.json file in the latest version. Maybe you changed something in your code that caused this?

cloudinary / php-transformation-builder-sdk