BugTrack/474

attach rev.1.34で添付に失敗†

• ページ: BugTrack
• 投稿者: Ratbeta
• 優先順位: 重要
• 状態: 完了
• カテゴリー: プラグイン
• 投稿日: 2003-10-11 (土) 18:25:43
• バージョン: 1.4rc4+CVS

メッセージ†

タイトルの通りですが、attach.inc.phpの最新版(rev.1.34)で、
アップロードに失敗します。

rev.1.33までではこの現象は発生しなかったので、
attach.inc.phpの問題かと思われます。

環境は、XREAサーバを使用しております。
OS : RedHat Linux 7.3
php : php 4.3.2 サーバ : Apache 1.3.27
PukiWiki : PukiWiki 1.4(CVS最新版))
また、ローカルサーバでも同じ現象が発生しました。
OS : WinXP SP1
php : php 4.3.4 RC1
サーバ : Apache 1.3.28
PukiWiki : PukiWiki 1.4(CVS最新版))

修正をお願いします。

• 確かにおかしな状況が発生することがあるようです。添付は実際にされているのですが、表示されない現象があるようです。こちらでも現象を調べてみますので そちらでもチェックしてみてください。全添付ファイルの表示でも出て来ませんでしょうか? merlin:Plugin/Demo/attach.inc.php -- merlin 2003-10-11 (土) 21:34:15
  • うーん 再現しなくなっちゃった。先ほどは 凍結したら 表示されなくなったんですが... ちなみにこちらのサーバは S34です。9/28版CVSにモジュールを最新に入れ換えたものですが(paraedit,preeditあたっているけど) -- merlin 2003-10-11 (土) 21:41:07
  • putstatus 時にも chmod がいるのかな? -- merlin 2003-10-11 (土) 21:45:40
• 全く問題なし。再現しません。プラットフォーム依存があるのでしょうか？ -- sha 2003-10-12 (日) 02:05:09
• 全添付ファイルの表示でも表示されません。 -- Ratbeta 2003-10-12 (日) 11:05:08
• ちなみに凍結は試してないので分かりません…。 -- Ratbeta 2003-10-12 (日) 11:08:00
  • http://wiki.tec-tech.org (S39) に 10/11 23:00の CVSを そのままいれて(attach.inc.phpは XREA パッチ付きに変更) みましたが ちゃんと添付できているようです。 http://wiki.tec-tech.org/?TEST ここで試してみていただけますか? *1

    ここの添付後のパーミッションは以下のようになっています。-- merlin 2003-10-12 (日) 12:46:19

    drwx---rwx   2 XXXXXX hpusers      4096 Oct 12 12:42 attach
    ....
    ncftp /public_html/attach > ls -la
    drwx---rwx   2 XXXXXX hpusers      4096 Oct 12 12:42 .
    drwx---r-x  16 XXXXXX hpusers      4096 Oct 12 12:22 ..
    -rw----r--   1 apache   apache      55787 Oct 12 12:42 54455354_72746C383133392E63
    -rw-r--r--   1 apache   apache          5 Oct 12 12:42 54455354_72746C383133392E63.log
    -rw----r--   1 apache   apache        953 Oct 12 12:41 54455354_CCB5C2EA2D31
    -rw----r--   1 apache   apache        953 Oct 12 12:39 54455354_CCB5C2EA2D31.1
    -rw-r--r--   1 apache   apache          5 Oct 12 12:41 54455354_CCB5C2EA2D31.log
    -rw----rw-   1 XXXXXX hpusers        29 Oct 12 12:10 dir.txt

• はっ... まさか attach.inc.ref 内の MAX_FILESIZEを 以前は 変えていたのが 今回は変えてないとか なんて ミスではないですよね? m(_._)m ファイルサイズオーバーの時って何かメッセージが出るといいのかもしれませんね -- merlin 2003-10-12 (日) 13:27:32
• 添付したファイルはどれも10KBにも満たない小さな物です。ちなみに↑のページでは添付できました。う～ん、謎です…。 -- Ratbeta 2003-10-12 (日) 20:03:15
  • 失礼いたしました。m(_._)m XREA広告ありページでは、.htaccessの設定での手動広告表示と attach.inc.php の XREA パッチが必須となりますので上記のテストページでは行っていますがいかがでしょうか?上記は、public_html 内設置ですがそちらはいかがでしょう? public_html/x/に広告手動で入れて設置したものはどうでしょうか? -- merlin 2003-10-12 (日) 20:15:13
  • localの apache 1.3.27でテストしましたがうまく行きます。Winとレンタルサーバ特有の問題かもしれません。 -- merlin 2003-10-12 (日) 20:21:49
  • 1.33と1.34のdiffをざっと見ましたが、特にenbugされてはいないと思います。今回追加になったのは、認証のサポートですが、添付しようとしているページは編集、閲覧の認証がかかってませんでしょうか? -- merlin 2003-10-12 (日) 21:04:33
  • まず、.htaccessの設定とXREAパッチの修正はしています。また、添付するページには何の認証も掛かっていません。 -- Ratbeta 2003-10-12 (日) 21:45:51
• 上記サイトに入れたファイルを添付しますので そちらのとdiff取ってチェックしてみてください。なお .htaccessは最低限のものになっています。 -- merlin 2003-10-12 (日) 22:03:49
  • puki.tar.gz
  • そちらのattach.inc.phpだと正常に添付できました。XREAのパッチの使い方がまずかっただけかも知れません。なお、.htaccessの設定は同じでした。一応これで解決という事に。(^^; -- Ratbeta 2003-10-13 (月) 19:33:26
  • よかったですぅ。実は、少しづつ かわってるんですよね。パッチにしちゃうと... うーん、XREA対応の設定ページ作ってファイルを置いた方がよさようですねぇ。win用のdiffツールの解説もいるのかなぁ? ところで、パーミッション関連の調整あたりも本体に組み込んだ方がいいのかなぁ? -- merlin 2003-10-13 (月) 21:07:40
• こんな感じにしておけばいいですかね。 -- ぱんだ 2003-10-18 (土) 18:03:28

  Index: attach.inc.php
  ===================================================================
  RCS file: /cvsroot/pukiwiki/pukiwiki/plugin/attach.inc.php,v
  retrieving revision 1.34
  diff -u -r1.34 attach.inc.php
  --- attach.inc.php	30 Sep 2003 07:08:43 -0000	1.34
  +++ attach.inc.php	18 Oct 2003 09:02:43 -0000
  @@ -35,6 +35,10 @@
   // アップロード/削除時にパスワードを要求する(ADMIN_ONLYが優先)
   define('ATTACH_PASSWORD_REQUIRE',FALSE); // FALSE or TRUE
   
  +// ファイルのアクセス権
  +define('ATTACH_FILE_MODE',0644);
  +//define('ATTACH_FILE_MODE',0604); // for XREA
  +
   // file icon image
   if (!defined('FILE_ICON'))
   {
  @@ -176,7 +180,10 @@
   	{
   		return array('result'=>FALSE,'msg'=>$_attach_messages['err_exists']);
   	}
  -	move_uploaded_file($file['tmp_name'],$obj->filename);
  +	if (move_uploaded_file($file['tmp_name'],$obj->filename))
  +	{
  +		chmod($obj->filename,ATTACH_FILE_MODE);
  +	}
   	
   	if (is_page($page))
   	{

• ありがとうございます。当面はこれかなぁ? global にして 全ファイルで読み込めるようにするといいけどそれは大変.. -- merlin 2003-10-19 (日) 00:53:20

• BugTrack
• 開発版
  • リポジトリ
  • 開発日記
• :CategoryDev
  • PukiWiki2
  • PukiWiki/1.5
  • PukiWiki/1.4
  • PukiWiki/1.3
  • メーリングリスト
  • 開発談義
  • 雑談
• 開発者向け情報
  • Books
  • プラグイン

最新の16件