![[PukiWiki]](image/pukiwiki-dev.png "[PukiWiki]")
attach プラグインはファイル情報を取得すると自動的に md5 でのハッシュ計算を行います。そのため、添付ファイルのサイズ、数によっては、添付ファイルの情報を含むページの表示に時間を要し、その間 cpu を消費し続けます。これは、使い勝手についての問題、および、システム運用上の負荷の問題となっています。問題が発生する条件は限られますが、少ない副作用で実現可能であるならば対処を行ってもよいように思います。
問題点の補足 2007/7/30 追記
md5 を計算する際に OS のディスクキャッシュが追い出され、環境によっては他のアプリケーションに影響を与えます（正式にはバッファキャッシュとかページキャッシュとか？）
簡潔に言えば md5 sum の表示は必要か？ ということになると思います。
評価の前提
コメント
利点
欠点
利点
欠点
コメント
利点
欠点
利点
欠点
コメント
パッチ*2を以下に示します。
ファイル名が非常に長いなどでハッシュファイルを作成できない場合、毎回計算が実行されます。
Index: attach.inc.php
===================================================================
--- attach.inc.php (revision 39)
+++ attach.inc.php (revision 41)
@@ -47,6 +47,9 @@
// mime-typeを記述したページ
define('PLUGIN_ATTACH_CONFIG_PAGE_MIME', 'plugin/attach/mime-type');
+// use md5 cache mechanizm
+define('PLUGIN_ATTACH_HASH_CACHE', TRUE); // FALSE or TRUE
+
//-------- convert
function plugin_attach_convert()
{
@@ -449,9 +452,67 @@
$this->logname = $this->basename . '.log';
$this->exist = file_exists($this->filename);
$this->time = $this->exist ? filemtime($this->filename) - LOCALZONE : 0;
- $this->md5hash = $this->exist ? md5_file($this->filename) : '';
+
+ $this->hashCacheFile =
+ CACHE_DIR . encode($page) . '_' . encode($this->file) .
+ ($age ? '.' . $age : '') . '.attach_hash';
+
+ if ( PLUGIN_ATTACH_HASH_CACHE ) {
+ $this->md5hash = $this->getHashCache();
+ } else {
+ $this->md5hash = $this->exist ? md5_file($this->filename) : '';
+ }
}
+ function getHashCache()
+ {
+ // cache file will be stored in cache/*.attach_md5 file.
+ // Format: 3 int and 1 string splited by \t
+ // cache version : int (0)
+ // size : int (byte)
+ // stamp : int (mtime)
+ // md5sum : stringn (hex text)
+
+ if ( ! $this->exist ) { return ''; }
+
+ $cache_version = 0;
+
+ // open target file.
+ $fpf = fopen( $this->filename, "r" );
+ if( ! $fpf ) { return ''; } // It's strange...
+ $fresh = fstat( $fpf ); // get stat info
+
+ // open hash cache file
+ $fpc = fopen( $this->hashCacheFile, "r" );
+ if( $fpc ) { // cache existing.
+ $cached = fscanf($fpc, "%d\t%d\t%d\t%s\n");
+ if ( $cached[0] == $cache_version &&
+ $cached[1] == $fresh['size'] &&
+ $cached[2] == $fresh['mtime'] ) { // fresh and fine cache. use it.
+ fclose( $fpc );
+ fclose( $fpf );
+ return $cached[3];
+ // return "cache:" . $cached[3];
+ }
+ fclose( $fpc ); // close cache file once.
+ }
+
+ $md5 = md5_file($this->filename);
+
+ // Need to create cache
+ $fpc = fopen( $this->hashCacheFile, "w" );
+ if ( ! $fpc ) { fclose( $fpf ); return $md5; }
+
+ // generate cache file
+ fwrite($fpc,
+ sprintf("%d\t%d\t%d\t%s\n",
+ $cache_version, $fresh['size'], $fresh['mtime'], $md5 ));
+ fclose( $fpc );
+ fclose( $fpf );
+ return $md5;
+ // return "gen:" . $md5;
+ }
+
// ファイル情報取得
function getstatus()
{
@@ -617,7 +678,10 @@
}
}
- // バックアップ
+ // remove hash cache.
+ if ( PLUGIN_ATTACH_HASH_CACHE ) { @unlink($this->hashCacheFile); }
+
+ // バックアップ
if ($this->age ||
(PLUGIN_ATTACH_DELETE_ADMIN_ONLY && PLUGIN_ATTACH_DELETE_ADMIN_NOBACKUP)) {
@unlink($this->filename);
$obj = & new AttachFile($refer, $file, $age); +$obj->md5hash = $obj->exist ? md5_file($obj->filename) : '';
-$this->md5hash = $this->exist ? md5_file($this->filename) : '';
$this->type = attach_mime_content_type($this->filename);これも、このgethash()の様にgettype()としてやるのが良いです。そして$this->typeの呼び出し箇所2カ所を$this->gettype()とします。
$obj = & new AttachFile($refer, $file, $age);
- return $obj->getstatus() ?
+ return $this->exist ?
$obj->info($err) :
array('msg'=>$_attach_messages['err_notfound']);
とするのが無難でしょうか？ -- 2008-05-24 (土) 11:39:02ぃぉぃぉさんのご指摘通り、attach_mime_content_type()内で処理される、下記の処理が、行われているため、大きなサイズのファイルを添付したとき、 数秒程度の負荷がかかるようです。
$size = @getimagesize($filename);
この関数は、どんなファイルであっても画像ファイルを前提にかなりの部分を読み込んでしまい、オーバーヘッドになっているようです。 画像拡張子を持たないファイルで、中身だけ画像であるファイルはまず使われないかと思います。
逆にjpgを装った、不正なファイルはあるかもしれないので、
画像拡張子を持ったファイルのみ中身をチェックするようにするのはいかがでしょうか。
実際には下記のコードを使用してみました。
... ここまで同じです。
if (! file_exists($filename)) return $type;
$matches = array();
if (! preg_match('/_((?:[0-9A-F]{2})+)(?:\.\d+)?$/', $filename, $matches))
return $type;
$filename_decode = decode($matches[1]);
$psinfo0 = pathinfo(decode($filename_decode));
$ext0 = $psinfo0['extension'];
if( preg_match( '/gif|png|jpg|jpeg/i', $ext0 ) ) {
$size = @getimagesize($filename);
if (is_array($size)) {
switch ($size[2]) {
case 1: return 'image/gif';
case 2: return 'image/jpeg';
case 3: return 'image/png';
case 4: return 'application/x-shockwave-flash';
}
}
}
// mime-type一覧表を取得
- if (preg_match("/\.$ext$/i", $filename)) return $_type;
+ if (preg_match("/\.$ext$/i", $filename_decode)) return $_type;
適用した結果、巨大ファイル(pdfなど）を添付した時でも、2秒程度の遅れがなく表示できるようになりました。
コード自体に改善点はあるかもしれませんが、参考になれば幸いです。 -- Thanks 2010-05-29 (土) 19:16:39
| Site admin: PukiWiki Development Team PukiWiki 1.5.4+ © 2001-2022 PukiWiki Development Team. Powered by PHP 8.2.12. HTML convert time: 0.218 sec. |
|