Azure / azure-storage-php

Microsoft Azure Storage Library for PHP
MIT License
217 stars 200 forks source link

Undefined class constant 'CONTENT_TYPE_LOWER_CASE' #276

Open Growiel opened 3 years ago

Growiel commented 3 years ago

Which service(blob, file, queue, table) does this issue concern?

Blob

Which version of the SDK was used?

1.5.2 of https://github.com/Azure/azure-storage-blob-php

What's the PHP/OS version?

PHP 7.3

What problem was encountered?

Error 500: Undefined class constant 'CONTENT_TYPE_LOWER_CASE'

Steps to reproduce the issue?

Have you found a mitigation/solution?

The constant CONTENT_TYPE_LOWER_CASE needs to be defined in the correct file (most likely Blob/Internal/BlobResources.php)

Is there a failing request ID related to this problem returned by server? What is it?

No.

You can see on the commit right here: https://github.com/Azure/azure-storage-blob-php/commit/2475330963372d519387cb8135d6a9cfd42272da#diff-cc5cc0b2d11a12bd4aeb556360e53e47f0872012b38521560d0627deedfe4156

That they changed the constant from CONTENT_TYPE to CONTENT_TYPE_LOWER_CASE the latter one doesn't exists.

This is urgent for us, our site is totally inacessible due to this error.

Thanks !

spaze commented 3 years ago

It was added in https://github.com/Azure/azure-storage-php/commit/d99ae09b0377f95b67bcb340ea39e2827fa1d56a, did you also update common lib to icrosoft/azure-storage-common 1.5.1 too?

algorhythm commented 2 years ago

@spaze This should be forced in composer.json of this package, isn't it?

spaze commented 2 years ago

@spaze This should be forced in composer.json of this package, isn't it?

It's not, the constraint would need to be ~1.5.1. It isn't and that's why this bug exists :-)