cloudinary / cloudinary_php

PHP extension for Cloudinary
https://cloudinary.com/documentation/php_integration
MIT License
389 stars 151 forks source link

Wrong CLOUDINARY_URL #331

Closed stebogit closed 3 years ago

stebogit commented 3 years ago

Describe the bug in a sentence or two

The provided CLOUDINARY_URL value on my dashboard has the following format, please note the dot between key and secret: cloudinary://[<key>.<secret>@]<cloud>

Screen Shot 2021-11-22 at 1 32 52 AM

However, Cloudinary\Configuration\ConfigUtils::parseCloudinaryUrl expects cloudinary://[<key>:<secret>@]<cloud>

Although I'm using an old version of the laravel package (1.0.0, which used the SDK v2.0.0-beta6), I'm using Laravel 5.6, the code is the same on the latest version of the PHP SDK.

Now, simply replacing the . with a : in the provided URL everything works as expected, however I wonder why the inconsistency?

Issue Type (Can be multiple)

Operating System

Environment and Frameworks (fill in the version numbers)

michalkcloudinay commented 3 years ago

Hi @stebogit, Thanks for raising this issue. We are going to implement a fix soon. I will update here once the fix is released.

michalkcloudinay commented 3 years ago

Hi @stebogit, A fix for this issue was released. Please let us know if it works on your end. Thanks

stebogit commented 3 years ago

@michalkcloudinay I see the CLOUDINARY_URL value has changed on my dashboard and it now follows the expected pattern. Thanks!