Skip to content

Update README.txt#621

Merged
rosell-dk merged 2 commits intorosell-dk:masterfrom
lukapaunovic:patch-1
Dec 22, 2025
Merged

Update README.txt#621
rosell-dk merged 2 commits intorosell-dk:masterfrom
lukapaunovic:patch-1

Conversation

@lukapaunovic
Copy link
Contributor

Description

Recent versions of WebP Express require an instance-specific hash parameter
when using WebP On Demand on NGINX.

Apache (.htaccess) rules include this automatically, but NGINX users must pass
the hash explicitly. If the hash is missing, NGINX may serve
webp-on-demand.php as a static file instead of executing it, resulting in a
PHP file download and errors such as:

Configuration file was not found (wod-options.<hash>.json)

This pull request updates the documentation to clearly explain the requirement,
documents where to find the hash, and adds a working NGINX configuration example
using an internal rewrite (rewrite ... last) to ensure proper PHP execution
on NGINX + PHP-FPM setups (including Plesk).

Forgot to close code tag
@rosell-dk rosell-dk merged commit d8d4427 into rosell-dk:master Dec 22, 2025
@lukapaunovic
Copy link
Contributor Author

@rosell-dk I edited the wrong file, i forgot to edit .md should I do it?

@rosell-dk
Copy link
Owner

I'll do it

@lukapaunovic
Copy link
Contributor Author

#623 done

@rosell-dk
Copy link
Owner

I have created a new release which provides a notification to Nginx users that uses rewrite rules

@lukapaunovic
Copy link
Contributor Author

lukapaunovic commented Dec 23, 2025

Can you confirm is it correct to use try files first and then if it doesn't work to use internal rewrite?
Is there any downside of using internal rewrite?
Try files was working with nginx additional directives in plesk. But after hash i tried adding hash but it only worked with internal rewrite.
It could be that reasons is PHP fpm and fastcgi.
That parameter can't be passed by try files but I'm not sure. What are your thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants