Skip to content

Conversation

@jinrenjie
Copy link

What was changed

Static files for the frontend SPA are now served directly via Nginx instead of being routed through the backend project. This removes the need for any SPA-specific configuration in the backend.

Why?

In the current version, refreshing on any frontend route other than the Home page results in a 404 response. By serving SPA routes directly with Nginx, we can handle routing at the web server level, simplifying backend configuration and improving performance.

Checklist

@butschster butschster self-requested a review September 8, 2025 07:04
@butschster butschster self-assigned this Sep 8, 2025
@butschster
Copy link
Member

Hey! Thx for the PR! Did you try 1.13.4-dev version?

@jinrenjie
Copy link
Author

@butschster I deployed it using a Docker image with one click. Are we going to abandon updating the Docker image and instead build it directly from the source code?

@butschster
Copy link
Member

@jinrenjie I released version buggregator 1.13.4-dev, where (if I’m not mistaken) I fixed the issue with navigating between pages. Did you have a chance to try it?

Regarding the Docker image — if I recall correctly, that’s how we used to do it before. Later we switched to handling HTML on the PHP side, since there are some additional limits there.

@jinrenjie
Copy link
Author

jinrenjie commented Sep 10, 2025

@jinrenjie I released version buggregator 1.13.4-dev, where (if I’m not mistaken) I fixed the issue with navigating between pages. Did you have a chance to try it?

Regarding the Docker image — if I recall correctly, that’s how we used to do it before. Later we switched to handling HTML on the PHP side, since there are some additional limits there.

@butschster I deploying with the ghcr.io/buggregator/server:1.13.4-dev image, the 404 error on page refresh still persists!
Screenshot 2025-09-10 at 15 58 55

Screenshot 2025-09-10 at 15 59 08

@jinrenjie jinrenjie closed this Sep 10, 2025
@jinrenjie jinrenjie reopened this Sep 10, 2025
@jinrenjie
Copy link
Author

If the frontend is to be served via PHP, is what I did in this buggregator/server#290 PR correct?

@butschster
Copy link
Member

If the frontend is to be served via PHP, is what I did in this buggregator/server#290 PR correct?

Good question! I'll check it. Thx!

@butschster
Copy link
Member

butschster commented Sep 11, 2025

Hi @jinrenjie 👋
I’ve already replied to your pull request buggregator/server#290 and left a review there. That one should be enough to fix the issue. If for some reason it doesn’t fully resolve the problem, then we can revisit this repository as a fallback. Thanks again for taking the time to work on this 🙌

@jinrenjie jinrenjie closed this Sep 11, 2025
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.

Why not serve SAP with Nginx?

2 participants