-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Prevent use of double sub-path and incorrect asset path in manifest #14827
Prevent use of double sub-path and incorrect asset path in manifest #14827
Conversation
MakeAbsoluteAssetURL should just url join the static url prefix on to appurl if it is not an absolute path - this is because StaticURLPrefix is an absolute prefix not a relative prefix to the app sub url. Fix go-gitea#14422 Signed-off-by: Andrew Thornton <art27@cantab.net>
Codecov Report
@@ Coverage Diff @@
## master #14827 +/- ##
=========================================
Coverage ? 42.23%
=========================================
Files ? 771
Lines ? 82181
Branches ? 0
=========================================
Hits ? 34711
Misses ? 41821
Partials ? 5649
Continue to review full report at Codecov.
|
@uli-heller can you also verify this patch fix it for you too? |
I'm about to verify the patch against master, hope this is fine:
|
MakeAbsoluteAssetURL should just url join the static url prefix on to appurl
if it is not an absolute path - this is because StaticURLPrefix is an absolute
prefix not a relative prefix to the app sub url.
Fix #14422
Close #14427
Signed-off-by: Andrew Thornton art27@cantab.net