-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Routing doesn't work when accessing CSV Importer at subdirectory of a domain #3295
Comments
Could you share your nginx configuration please? |
I am having a similar issue with the app in general. My problem is when I have an nginx reverse proxy that exposes the whole of Firefly III on Here's the proxy I use (might be very wrong, would love suggestions):
|
This is my Nginx Configuration for both Firefly and CSV Importer accessible at one subdomain. I'm able to access every website of the CSV Importer at the
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Bug description
I am running Firefly III version 1.0.5 (Docker/develop), and my problem is:
I#m trying to access the CSV Importer at the Public Domain
firefly.mydomain.tld/csv
via a Nginx Reverse Proxy.I'm able to access the index page of the CSV Importer at
firefly.mydomain.tld/csv
, and the corresponding JS and CSS files by setting the environment variable inside the Docker Container like this:ASSET_URL=firefly.mydomain.tld/csv
.But then, on loading the index page, an instant redirect to
firefly.mydomain.tld/token
instead offirefly.mydomain.tld/csv/token
happens because theroute()
function calls insideresources/views/index.twig
return the wrong URLs.Steps to reproduce
Access CSV Importer on subdirectory of a Domain.
Extra info
Setting the environment like this
APP_URL=firefly.mydomain.tld/csv
didn't work, since Route seems to not respect this environment variable(#18613).Maybe grouping and prefixing with a dynamic prefix adjustable through environment variables could be a possible solution.
The text was updated successfully, but these errors were encountered: