ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 13 years ago

Closed 13 years ago

#558 closed enhancement (fixed)

ファイルアップロード機能の管理面向上

Reported by: takanashi Owned by: takanashi
Priority: minor Milestone: OpenPNE2.8Beta2
Component: core【OpenPNEコア部分】 Version:
Keywords: Cc:

Description

1)管理画面からファイル削除できるようにする
2)トピックの削除契機でのファイル削除を行う
3)トピックの編集・削除時の挙動を画像にあわせる
4)元ファイル名をDBに入れる段階で、ヘッダの文字列チェックを厳しくする

Change History (19)

comment:1 Changed 13 years ago by takanashi

Milestone: OpenPNE2.8RC2OpenPNE2.8RC1
Owner: changed from nobody to ebihara

comment:2 Changed 13 years ago by ebihara

  • 1) 2) はファイルの削除関数を作って、それをコールするような形にして対応します
  • 3) に関しても1) 2)で作成した関数をコールしますが、こちらは画像に合わせての調節をします
  • 4) についてはセキュリティ面での修正ですので、修正後に詳述します

comment:3 Changed 13 years ago by takanashi

追加:ファイルアップロード機能自体のオン/オフ設定をconfig.phpに追記。
offの時には直たたきでアップロードしようとしても弾くようにする。
管理画面のファイルアップロード機能の取り外しも行う。

comment:4 Changed 13 years ago by ebihara

1)は確認画面も挟みます(画像削除と同一の挙動)

comment:5 Changed 13 years ago by ebihara

ファイルアップロード機能のON/OFFを設定できるようにしました。

comment:6 Changed 13 years ago by ebihara

ファイルアップロード機能のON/OFFを設定できるようにしました。

r2237です

comment:7 Changed 13 years ago by ebihara

3)ではそのファイルをリンクしている箇所も削除します。

comment:8 Changed 13 years ago by ebihara

r2238で管理画面からファイルを削除できるようにしました。

comment:9 Changed 13 years ago by ebihara

r2239でトピック削除時にファイルも削除するようにしました

comment:10 Changed 13 years ago by ebihara

3)で実装するものは以下

  • 管理画面からのファイル削除時、そのファイル投稿元のトピックにあるリンクを削除
  • トピック編集時にもファイルアップロードができるように

comment:11 Changed 13 years ago by ebihara

3)でやるもの追加

  • トピック編集時、ファイルの削除ができるように

comment:12 Changed 13 years ago by ebihara

r2248, r2249, r2250で3)の対応を行いました。

comment:13 Changed 13 years ago by ebihara

r2252 コミットし忘れです。3)の対応です。

comment:14 Changed 13 years ago by ebihara

コミットし忘れはr2251でした。

comment:15 Changed 13 years ago by ebihara

r2252

HTTP Response Splitting対策です。

PHP4.4.2もしくはPHP 5.1.2以降を使用している場合はこの攻撃の影響を受けることはありませんが、改行文字が含まれうるc_file.original_filenameをヘッダに埋め込む際、改行文字列を除去し、古いバージョンのPHPを使用していた場合にもこの脆弱性の影響を受けないように修正しました。

comment:16 Changed 13 years ago by ebihara

r2253 ファイルアップロード時にHTTP POSTによりアップロードされたファイルかどうかを調べるようにしました。

comment:17 Changed 13 years ago by ebihara

Owner: changed from ebihara to takanashi

一通りチェックしましたが、こんなところでしょうか。ご確認よろしくお願いします。

comment:18 Changed 13 years ago by takanashi

確認します。

comment:19 Changed 13 years ago by takanashi

Milestone: OpenPNE2.8RC1OpenPNE2.8Beta2
Resolution: fixed
Status: newclosed

2.8beta2でリリース

Note: See TracTickets for help on using tickets.