Hardcoded Links locks enduser into web root about maraschino HOT 7 CLOSED

Hi,

Is the following post related to your issue?

[Req] urlbase/web_root/http_root ability:
#27

from maraschino.

Looks like it's a similar issue

from maraschino.

url_for can still be used/replaced in all the html templates and python thats not a big deal.. The problem is all the hard coded linking in the jquery and less files. I have no idea how to get around that other than inline styling and scripting which i highly doubt will happen.

from maraschino.

It's been a long time coming but i just merged this functionality. Please have a play around with it and report back with any feedback. Note that to run on port 80 you will need to grant Maraschino 'sudo' privileges.

from maraschino.

I've tried the new --webroot argument for Maraschino, but it seems that something is wrong with the way the less files pick it up. I see that a webroot.less file is created, but the contents seem to be wrong. I had to apply the following change before it would work for me. After that it is fine. If this change is correct, I will create a pull request for you to apply it.

index 4434cda..cf1d67b 100644
--- a/maraschino/__init__.py
+++ b/maraschino/__init__.py
@@ -120,7 +120,7 @@ def initialize():
         # Set up webroot for .less
         less_webroot = os.path.join(RUNDIR, 'static/less/webroot.less')
         f = open(less_webroot, 'w')
-        f.write('@webroot: "http://127.0.0.1:%i%s";' % (PORT, WEBROOT))
+        f.write('@webroot: "../../../%s";' % (WEBROOT))
         f.close()

         # Set up the updater

from maraschino.

I just noticed the seperate webroot_relative branch, and those changes seem to work properly for me.

from maraschino.

cool, i was just about to point you to that branch :)

from maraschino.