Issue information
-
#035073
-
Cannot Reproduce
-
2.5.0
-
-
-
0 - None Assigned
Issue Confirmations
-
Yes (0)No (0)
1
This.. May not REALLY be a bug.. But.. At best, It's confusing..
Ok.. So, we set that as ./downloads
Everything on the front end of the forum worked great.. No problems.. Happy as can be.
BUT.. user went to import from a ZIP file, and it failed.
Removing the suppression in classFtp.php got me the first error there. So.. Basically.. It seems that in the ACP.. It's none too happy with having ./downloads as the path.. Changed it to just /downloads, and it works fine..
But.. What happens if the path is ../downloads? that's where the main concern is.. No big deal if it's ./downloads and we can use /downloads and have it work.. But if ../ doesn't work.. That's a problem.
Remote storage: path to downloads Relative to the home directory for the FTP user Example: ../private_files/downloads
Ok.. So, we set that as ./downloads
Everything on the front end of the forum worked great.. No problems.. Happy as can be.
BUT.. user went to import from a ZIP file, and it failed.
Warning: ftp_chdir() [function.ftp-chdir]: Can't change directory to //./downloads/./downloads: No such file or directory in /public_html/ips_kernel/classFtp.php on line 122
Fatal error: Uncaught exception 'Exception' with message 'CHDIR_FAILED' in /public_html/ips_kernel/classFtp.php:131 Stack trace: #0/public_html/admin/applications_addon/ips/downloads/modules_admin/tools/bulkimport.php(868): classFtp->chdir('./downloads') #1 /public_html/admin/applications_addon/ips/downloads/modules_admin/tools/bulkimport.php(440): admin_downloads_tools_bulkimport->_processFiles(Array, Array, Array) #2/public_html/admin/applications_addon/ips/downloads/modules_admin/tools/bulkimport.php(119): admin_downloads_tools_bulkimport->zipIndexAdd() #3 /public_html/admin/sources/base/ipsController.php(302): admin_downloads_tools_bulkimport->doExecute(Object(ipsRegistry)) #4 /public_html/admin/sources/base/ipsController.php(120): ipsCommand->execute(Object(ipsRegistry)) #5 /public_html/admin/sources/base/ipsController.php(65): ipsController->handleRequest() #6 /public_html/admin/index.php(26): ipsControl in /home/aikencen/public_html/ips_kernel/classFtp.php on line 131
Removing the suppression in classFtp.php got me the first error there. So.. Basically.. It seems that in the ACP.. It's none too happy with having ./downloads as the path.. Changed it to just /downloads, and it works fine..
But.. What happens if the path is ../downloads? that's where the main concern is.. No big deal if it's ./downloads and we can use /downloads and have it work.. But if ../ doesn't work.. That's a problem.
Not verified. If it works on the front end fine, clearly there's some difference between front end and ACP we should be able to resolve.
Are you sure it worked on the front end fine? There is literally no difference in the command being used that I can see between the ACP and the front end, however the front end does wrap those commands in a try/catch block to prevent the exception from hitting the screen (which I've added to the ACP).
My guess is, it was failing silently on the front end and no one noticed. If you hit this again, send the ticket up for further investigation.
My guess is, it was failing silently on the front end and no one noticed. If you hit this again, send the ticket up for further investigation.
0 user(s) are reading this issue
0 members, 0 guests, 0 anonymous users
