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

Allow app running under subpath #24244

Merged
merged 3 commits into from
Oct 10, 2024
Merged

Conversation

vladimir-kotikov
Copy link
Contributor

@vladimir-kotikov vladimir-kotikov commented Oct 6, 2024

This is currently impossible as the WS server is only accepting requests on /api url, plus serve-static is pretty straightforward about serving files and can not be configured with a base directory. This change allows to set a base subdirectory for the Z2M app including websocket listener and rewrites request urls when serving static files so that they are resolved correctly.

@vladimir-kotikov vladimir-kotikov force-pushed the app_base_url branch 3 times, most recently from a53d3cc to aee23bb Compare October 6, 2024 20:39
@vladimir-kotikov
Copy link
Contributor Author

Just realized there's a #19911 which does pretty much the same, although I noticed there's no changes for the websocket url so I'm not quite sure how's that supposed to work.

@Koenkk
Copy link
Owner

Koenkk commented Oct 7, 2024

Could you also update the settings schema?

@vladimir-kotikov
Copy link
Contributor Author

Done!

@Koenkk Koenkk merged commit 1fdf0a0 into Koenkk:dev Oct 10, 2024
11 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Oct 10, 2024

Thanks!

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