mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-13 11:19:37 +03:00
Relative paths for local images #919
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @jelle82 on GitHub (Nov 21, 2018).
Make the paths to locally stored images relative instead of absolute. For example
<img src="/uploads/images/gallery/cat.png" />instead of<img src="http://bookstack.example.com/uploads/images/gallery/cat.png" />,It increases portability. f you want to change the domain name for the Bookstack App there is no need to change all the image urls afterwards.
@Sibbo07 commented on GitHub (Nov 21, 2018):
That would be very helpfull!
@ssddanbrown commented on GitHub (Nov 22, 2018):
This would be nice but will make the logic more complex.
Was a while ago but I think i decided to use full paths since we'd have to handle full paths anyway for external storage systems (S3). Easier to standardise on one format.
@andreicocari commented on GitHub (Jan 15, 2019):
Would it make sense in that case to have a "base domain" field in the Settings page? I believe Wordpress has a similar approach.
@ssddanbrown commented on GitHub (Jan 15, 2019):
@andreicocari Yeah, Although we essentially already have this via
APP_URLin the.envfile.@recklessnl commented on GitHub (Mar 5, 2019):
+1 for this!
@comnam90 commented on GitHub (Mar 14, 2019):
+1 for this, it's needed for environments where the internal and external URLs are different.
@GarethFox commented on GitHub (Mar 25, 2019):
+1 too ^^
I face the same issue using a temporary url to create content and modifying the url when staging to production environment with final url, all the images are not displayed any more, they are referring to the old url in use when uploaded.
@antontkv commented on GitHub (May 19, 2019):
+1
Have the same issue after migrating to another domain.
@Stefansen commented on GitHub (Jun 28, 2024):
any update on this issues?
@reingit commented on GitHub (Nov 26, 2025):
I fixed it like this for now:
Havent really tested it much, but everything I needed relative is relative now. Might crontab this in the future.
@ssddanbrown commented on GitHub (Nov 27, 2025):
@GarethFox Just be aware that this can mess with some functionality (like references, ZIP export handling) since the base URL is used as an anchor point for parsing/discovery, and it'd make it tricky to change URLs (especially sub-paths) if needed in the future.