将上传的图像放在公共文件夹中是否安全?

我刚刚和我的队友讨论了用户上传的图片在图片库中的位置。我想对我们建议的方法有更广泛的了解。

我的队友写了一个控制器+动作，调用file_get_contents上的图像文件放在一个文件夹，不可用于公共浏览(即，在服务器上的public_html之外)，并通过一个头回显它。这是安全的，但是由于我们使用的是Zend框架，所以它的爬行速度也很慢——由于执行bootstrap的查询，每次调用图像控制器都会花费我们大约500ms的延迟。这很烦人，因为图片库视图同时显示超过20张图片。

简而言之，相关代码为:

class ImageController extends Zend_Controller_Action {
    public function showAction () {
        $filename = addslashes($this->_getParam('filename'));
        if(!is_file($filename)) {
            $filename = APPLICATION_PATH.'/../public/img/nopicture.jpg';
        }
        $this->_helper->viewRenderer->setNoRender(true);
        $this->view->layout()->disableLayout();
        $img = file_get_contents($filename);
        header('Content-Type: image/jpeg');
        $modified = new Zend_Date(filemtime($filename));
        $this->getResponse()
             ->setHeader('Last-Modified',$modified->toString(Zend_Date::RFC_1123))
             ->setHeader('Content-Type', 'image/jpeg')
             ->setHeader('Expires', '', true)
             ->setHeader('Cache-Control', 'public', true)
             ->setHeader('Cache-Control', 'max-age=3800')
             ->setHeader('Pragma', '', true);
        echo $img;
    }
}

然后，在视图中，我们只需调用:

<img src="<?php echo $this->url(array('controller' => 'image', 'action' => 'show', 'filename' => PATH_TO_HIDDEN_LOCATION.'/filename.jpg')); ?>" />

我有一个不同的方法:我更喜欢保持原始图像在一个隐藏的位置，但一旦他们被要求，将它们复制到一个公共位置，并提供一个链接到它(与一个额外的机制，由cron运行，擦除公共图像目录每隔一段时间，为了不浪费空间，和一个robots.txt告诉谷歌不要索引目录)。该解决方案将文件(在每个给定时刻都有几个)放在一个可公开访问的目录中(前提是知道文件名)，但也只需要一个视图帮助器，因此不会启动bootstrap:

class Zend_View_Helper_ShowImage extends Zend_View_Helper_Abstract {
    public function showImage ($filename) {
        if (!file_exists(PUBLIC_PATH."/img/{$filename}")) {
            if (!copy(PATH_TO_HIDDEN_FILES."/{$filename}",PUBLIC_PATH."/img/{$filename}"))
                $url = PUBLIC_PATH.'/img/nopicture.jpg';
            else
                $url = PUBLIC_PATH."/img/{$filename}";
        } else {
            $url = PUBLIC_PATH."/img/{$filename}"
        }
        return "{$url}";
    }
}

在这个帮助器的帮助下，视图中的调用非常简单:

<img src="<?php echo $this->showImage('filename.jpg'); ?>" />

问题:我的方法是否像我的同事所说的那样构成安全威胁?这样做的潜在风险是什么?而且，最重要的是，安全威胁(如果有的话)是否超过了页面加载时间增加的10秒?

如果它很重要:我们正在开发一个拥有大约15K注册用户的社区门户，其中画廊是一个非常常用的功能。

*我粘贴的代码是经过编辑的，简化了我们每个人的想法的版本——只是为了展示两种方法的机制。

<a href="image.php?image=foobar.jpg&width=320&height=240" />

<img src="image.php?file=myFile.jpg&list=12345" />

echo $this->thumb(array('url' => HIDDEN_DIR . 'foo/bar.jpg'));

/_cache/thumbs/3858f62230ac3c915f300c664312c63f.jpg