goldsky / FileDownload-R

FileDownload snippet for MODX revolution
12 stars 7 forks source link

Download links throw 404 #27

Closed bonfish closed 9 years ago

bonfish commented 10 years ago

Hi!

Snippet call [[!FileDownload? &getDir=[[++base_path]]some/path/]] is in the start document. It loads and shows all files correctly. But when I try to download something when accessing it from the base_url (http://my.site), I see the issue form the subject.

When I load http://my.site/index.php?id=1 - everything works fine. All the pathes are equal on both pages. MODX Revo 2.2.11. Tested on localhost (Ubuntu server 12.04.3) and on Hetzner VPS.

The faulty link: http://bonflash.it.cx/modx-revo-test/ The working link: http://bonflash.it.cx/modx-revo-test/index.php?id=1

Thanks!

goldsky commented 10 years ago

How big is the file? On Jan 28, 2014 4:57 PM, "bonfish" notifications@github.com wrote:

Hi!

Snippet call [[!FileDownload? &getDir=[[++base_path]]some/path/]] is in the start document. It loads and shows all files correctly. But when I try to download something when accessing it from the base_url ( http://my.site), I see the issue form the subject.

When I load http://my.site/index.php?id=1 - everything works fine. All the pathes are equal on both pages. MODX Revo 2.2.11. Tested on localhost (Ubuntu server 12.04.3) and on Hetzner VPS.

The working link: http://bonflash.it.cx/modx-revo-test/ The faulty link: http://bonflash.it.cx/modx-revo-test/index.php?id=1

Thanks!

Reply to this email directly or view it on GitHubhttps://github.com/goldsky/FileDownload-R/issues/27 .

bonfish commented 10 years ago

@goldsky as seen on the pages from the links, there are two in the folder: 370 KB and 2.3 MB. Tried with pictures of 2-3 KB - the outcome is the same.

bonfish commented 10 years ago

I found out a little more on this. The error handler was initially set to the home page (id1). If I change the 404 handler from the it to something else (say, id2), the links from the 'faulty link' (site root, http://bonflash.it.cx/modx-revo-test/ ) won't give the subj error, but the 404 handler (id2) instead.

So, the memory problem is due to eternal requesting the 404 handler from itself. And it is not about some wrong memory usage, but links to files not working on the same pages

Hope this helps.


Update: found the same issue closed. Tried to reinstall the package - still the same error.

When installing or reinstalling, I get "Could not load package metadata for package filedownload." warning/error. On both instances: local and Hetzner. Other packages so far don't throw this warning.

bonfish commented 10 years ago

Works fine with friendly URL enabled. Might be something with DL link check on click, comparing them with the current page URL(site_url vs site_url/index.php?id=_id_)

goldsky commented 10 years ago

I've just updated the package to 1.1.8-pl If the problem is resolved, pls close this issue. Thx.

goldsky commented 9 years ago

I'm closing this issue. If problem persists, please post back.