Incorrect URL generated when Public ID of MediaAsset ends with dot + 1-5 characters

[holtkamp]

Incorrect URL generated when Public ID of MediaAsset ends with dot + 1-5 characters

[const-cloudinary]

Hello @holtkamp .

The goal of the extension setter is to replace/set the extension of the file, no matter what it was before.

In some cases it's hard to know whether it is a file extension or just a part of the filename (like in examples you provided).

If you want to build the right URLs, you can include the desired file extension in the constructor, like:
$mediaAsset = new CloudinaryMediaAsset('myImage.1234.jpg', $cloudinaryConfiguration);
echo mediaAsset->toUrl(); //https://res.cloudinary.com/accountName/image/upload/myImage.1234.jpg
And you can further manipulate it the way you want:
$mediaAsset->setExtension('png');
echo mediaAsset->toUrl(); //https://res.cloudinary.com/accountName/image/upload/myImage.1234.png

Or you can use transformations instead, it will deliver you file in the right format:
$mediaAsset = new CloudinaryMediaAsset('myImage.1234', $cloudinaryConfiguration);
$mediaAsset->format('png');
echo mediaAsset->toUrl(); //https://res.cloudinary.com/accountName/image/upload/f_png/myImage.1234

[holtkamp]

Hi @const-cloudinary, thanks for the swift response.

We intentionally store the PublicIDs of uploaded assets without extension to make it format-agnostic and stimulate serving it in whatever format is required for a certain use case.

As you suggested: by appending the extension to the PublicIDs and "not setting the extension afterwards", the problem can be circumvented, thanks!

I would suggest to document this "behaviour" / guidelines for the PublicID? Something like this:

When using dots '.' in the end of a public ID, note that the 1-5 characters after the dots are parsed as being the extension of the Media asset:

- myPublicId.1      // Extension: "1"
- myPublicId.12     // Extension: "12"
- myPublicId.123    // Extension: "123"
- myPublicId.1234   // Extension: "1234"
- myPublicId.12345  // Extension: "12345"
- myPublicId.123456 // No extension

 This derived extension will get overwritten when explicitly assigning it.

[dannyv-cloudinary]

Hi @holtkamp .

Thanks for the suggestion. I'll raise it to our documentation team for consideration :)

I'm going to close off this issue, but feel free to get in touch with us if you need anything further.

Thanks,
-Danny
Cloudinary Support