-
-
Notifications
You must be signed in to change notification settings - Fork 220
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
PyWB 2.6.2 ignoring SCRIPT_NAME when deployed under a prefix #686
Comments
For context, I had to change template escaping when going from PyWB 2.5.x to 2.6.x (see here) but otherwise the UKWA extension code is unchanged. I did not have to set the static path before, and I don't know how to set it. |
The only thing I can find is: pywb/pywb/rewrite/templateview.py Line 325 in e64e58f
But I can't set Also, I think the default should be Setting |
Ah, okay, so there's a difference between different pages. The actual playback pages are working fine, and pick up the right prefix automatically, and also pick up the It's the 'index' pages that have problems. e.g. the page that lists collections has no CSS/JS, and the Collection Search page too. But the Calendar and the playback itself are fine. |
- ensure pywb.static_prefix includes pywb.app_prefix (via SCRIPT_NAME) for non replay templates, fixes #686
Should be fixed in 2.6.3, should not need to set the static path or any additional changes. |
Describe the bug
Even after 2.6.2 #682, static resources are not being loaded when PyWB is deployed under a prefix. We deploy PyWB under e.g.
/wayback/
so the static resources are at/wayback/static/
. For 2.5.x this is picked up by theSCRIPT_NAME
UWSGI parameter, but this is ignored by 2.6.2.EDIT: Ah, so this is not really to do with static resources - the whole app does not realize it is deployed under a prefix and doesn't work at all for us.
Config is at:
Steps to reproduce the bug
Deploy under a prefix
Expected behavior
Application uses supplied prefix as the deployment prefix.
Screenshots
Environment
Based on container version
webrecorder/pywb:2.6.2
Additional context
The text was updated successfully, but these errors were encountered: