AssetManager: css merging not properly handling relative paths containing . or .. #1682

Closed
robocoder opened this Issue Sep 8, 2010 · 3 comments

1 participant

@robocoder

I installed piwik in a directory, 2 levels down from the web server's document root, i.e., /var/www/one/two/, and I'm getting a number of 404 errors, e.g., GET /themes/default/images/xxx, when it should be /one/two/themes/default/images/xxx.

In asset_manager_global_css.css, I see:

url(../../themes/default/./images/xxx)

This should be either:

url(./themes/default/images/xxx)
url(themes/default/images/xxx)
@robocoder

(In [3101]) fixes #1682

@robocoder

(In [3102]) refs #1682 - path consistency

@robocoder

(In [3104]) refs #1682 - use realpath to remove ..

@robocoder robocoder added this to the Piwik 1.1 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment