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

Base URI broken after upgrade #33

Open
nicofrand opened this issue Apr 17, 2019 · 4 comments

Comments

Projects
None yet
2 participants
@nicofrand
Copy link

commented Apr 17, 2019

Hi,

since the last upgrade (e5bc9f8) my install was broken. All the base URIs became:

# Should begin and end with a "/"
define("BAIKAL_CARD_BASEURI", "//card.php/");

# Should begin and end with a "/"
define("BAIKAL_CAL_BASEURI", "//cal.php/");

# Should begin and end with a "/"
define("BAIKAL_DAV_BASEURI", "//dav.php/");

whereas the previous and correct values are:

# Should begin and end with a "/"
define("BAIKAL_CARD_BASEURI", "/card.php/");

# Should begin and end with a "/"
define("BAIKAL_CAL_BASEURI", "/cal.php/");

# Should begin and end with a "/"
define("BAIKAL_DAV_BASEURI", "/dav.php/");

i guess the __PATH__ is equal to / here: https://github.com/YunoHost-Apps/baikal_ynh/blob/master/conf/config.system.php#L42-L48.

I am using the app on its own subdomain, ie. baikal.mydomain.tld, not on a subpath.

@nicofrand

This comment has been minimized.

Copy link
Author

commented Apr 17, 2019

Many thanks for the app BTW!

@Gofannon

This comment has been minimized.

Copy link

commented Apr 17, 2019

On my server, Baikal is installed on a subdirectory and runs

domain: https://example.org/baikal/

config screenshot
image

@nicofrand

This comment has been minimized.

Copy link
Author

commented Apr 17, 2019

@Gofannon This is not a subdomain then? This is a subdirectory?

@Gofannon

This comment has been minimized.

Copy link

commented Apr 18, 2019

@Gofannon This is not a subdomain then? This is a subdirectory?

Yes indeed !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.