rsoury / wp-imgix

Simple WordPress ImgIX Plugin - For use with S3-Uploads
13 stars 5 forks source link

Doesn't work without signing URLs #4

Closed mgtcampbell closed 1 year ago

mgtcampbell commented 1 year ago

Hi there,

In the docs it says:

If you're signing urls, Add define( 'WP_IMGIX_SIGNING_TOKEN', 'abcdefg123456' ) to your wp-config.php file.

Implying that this setting is optional.

However if I do not define this in my wp-config.php file, then I get the following error:

Fatal error: Uncaught Error: Undefined constant "WP_IMGIX_SIGNING_TOKEN" in /var/www/vhosts/ply.digital/urban.staging.ply.digital/wp-content/plugins/wp-imgix-master/includes/class-wp-imgix.php:61

If I then try and define this as null - i.e. define( 'WP_IMGIX_SIGNING_TOKEN', '' ) or define( 'WP_IMGIX_SIGNING_TOKEN', null), then the image URLs in the content are not replaced.

Please help.

Ideally I do not want to turn on signing URLs for this website.

Cheers Mark

mgtcampbell commented 1 year ago

FWIW it doesn't seem to work with signing URLs either... may just be a PHP 8 compatibility issue based on further error logging.

mgtcampbell commented 1 year ago

My bad - turned out there was an issue with the custom theme code