This repository was archived by the owner on Jan 7, 2026. It is now read-only.
Conversation
d7415
approved these changes
May 27, 2025
Contributor
d7415
left a comment
There was a problem hiding this comment.
As far as I can tell it will still serve the static files directly, it will just now log it.
Not sure why this was affecting the URL (unless there's a weird default I'm not seeing, or Laravel's doing something screwy, which wouldn't be entirely new).
But sure.
Co-authored-by: Martin Stone <1611702+d7415@users.noreply.github.com>
Member
Author
When trying to access signed url, we no longer directly go to the file, we use Lychee as proxy but the ending is still .jpg etc... With that image chunk, the registrations links works (use signature) but the images do not. :| |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is modifications of the nginx to take care of the annoying addition of the path into the url.
This had the nasty effect of breaking:
Tested locally to resolve the above mentioned issued.
We no longer also directly try to serve the static files/images as they lose their query string for some unknown reasons, (honestly, I am done digging)
Source of the nginx config: https://laravel.com/docs/12.x/deployment#nginx
This pull request updates the
default.conffile for the web server configuration, focusing on improving compatibility, security, and maintainability. The most important changes include adding IPv6 support, refining static file handling, and improving PHP request processing.Compatibility improvements:
listen [::]:80;in the server block.Static file and request handling:
/favicon.icoand/robots.txt, disabling access logs and "not found" logs for these files.ifcondition withtry_filesin the root location block.PHP processing improvements:
$realpath_rootforSCRIPT_FILENAMEand addedfastcgi_hide_header X-Powered-Byto enhance security.fastcgi_passdirective to use the correct path (/var/run/php/php8.4-fpm.sock) for PHP-FPM.