Skip to content
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

Route prefix is not supported #290

Open
mkahvi opened this issue Sep 8, 2024 · 5 comments
Open

Route prefix is not supported #290

mkahvi opened this issue Sep 8, 2024 · 5 comments
Assignees
Labels
question Further information is requested

Comments

@mkahvi
Copy link

mkahvi commented Sep 8, 2024

For example:

GET https://host/systems/alienrpg/images/IndBackground3.png 403 (Forbidden)

When it should be trying to access

https://host/myRoutePrefix/systems/alienrpg/images/IndBackground3.png

Problem is caused by starting paths with slash, forcing them to exit the route prefix scope.

So the asset path is defined as /systems/alienrpg/images/IndBackground3.png
When it should be systems/alienrpg/images/IndBackground3.png
Though this one since this is in CSS, it needs to be relative to the CSS file

Tested on version 3.1.8 since I can't update to Foundry v12 yet.

@pwatson100
Copy link
Owner

pwatson100 commented Sep 8, 2024

What O/S is the server running on?

@mkahvi
Copy link
Author

mkahvi commented Sep 8, 2024

Win11, that doesn't matter for route prefix problem.

Also seen here with the panicked with the image path being broken due to it:
route-prefix

^ I can fix that by removing the initial slash from the image path. But needing to fix it every time is not great.

@pwatson100
Copy link
Owner

I have tested V3.1.8 on my Win 11 system (running node.js) and am not seeing an issue. The image paths are valid as per Foundry VTT:
/systems/alienrpg/images/panic.webp

image
image

image

I am not sure where myRoutePrefix is coming from on your installation?

Just to make you aware I am not making any updates or changes to V3.1.8.

@pwatson100 pwatson100 added the question Further information is requested label Sep 9, 2024
@pwatson100 pwatson100 self-assigned this Sep 9, 2024
@mkahvi
Copy link
Author

mkahvi commented Sep 9, 2024

Ref: https://foundryvtt.com/article/configuration/

Fill in routePrefix in options.json and use the electron client to easily test how route prefix changes things.

@pwatson100
Copy link
Owner

OK. I am on the same page now. I have corrected this in the next version but that will only support FVTT V12 I am afraid.
Thank you for bringing it to my attention.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants