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

我的队友编写了一个 Controller + 操作，它调用 file_get_contents 放置在不可用于公共(public)浏览的文件夹中的图像文件(即，在服务器上的 public_html 之外) , 并通过标题回显它。这是安全的，但由于我们使用 Zend Framework，它的爬行速度也很慢 - 由于执行 Bootstrap 的查询，每次调用图像 Controller 都会花费我们大约 500 毫秒的延迟。这很烦人，因为图片库 View 同时显示 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;
    }
}

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

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

我有一个不同的方法:我更喜欢将原始图像保存在一个隐藏的位置，但是一旦被请求，就将它们复制到一个公共(public)位置并提供一个链接(有一个额外的机制，由 cron 运行，为了不浪费空间，不时地删除公共(public)图像目录，以及 robots.txt 告诉 Google 不要索引该目录)。该解决方案将文件(每个给定时刻一些文件)放在一个可公开访问的目录中(前提是知道文件名)，但也只需要一个 View 助手，因此不需要启动 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}";
    }
}

有了这个helper的帮助， View 中的调用就很简单了:

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

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

以防万一:我们正在开发一个拥有大约 15,000 名注册用户的社区门户网站，画廊是一个非常常用的功能。

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

最佳答案

I have a different approach: I prefer to keep the original images in a hidden location, but as soon as they are requested, copy them to a public location and provide a link to it

创造力+1。

Does my approach pose a security threat, as my coleague states? What are the potential risks of this? And, most importantly, do the security threats, if any, outweigh the 10 seconds gain on page load?

有点。是的，如果您有只允许某些人看到的图像，并且您将它们放在一个可公开访问的目录中，那么其他人可以看到该图像就会发生变化，这似乎是不可取的。我也不认为(可能是错误的)它会在页面加载时增加 10 秒，因为您必须复制图像，这是一个相当密集的操作，比使用 file_get_contents 或 readfile( ) 还要多。

This is secure, but since we use Zend Framework, it's also crawling slow - each call to the image controller costs us approx 500ms of lag due to the bootstrap's queries being executed.

如果我可以建议；针对此特定案例的 nuke Zend Framework。我也在为一个相当大的网站使用 Zend Framework，所以我知道 Bootstrap 可能需要比你想要的更长的时间。如果您绕过 Zend Framework，选择 vanilla PHP，这将显着提高性能。

此外，使用 readfile()，而不是 file_get_contents()。一个很大的区别在于 file_get_contents 会在输出之前将整个文件加载到内存中，而 readfile 执行此操作的效率更高。

关于php - 将上传的图像放在公共(public)文件夹中是否安全？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/6622854/