セキュリティアップデート

セキュリティアップデート

- Haruhiko Okumura の投稿
返信数: 3
何やらいろいろセキュリティアップデートがあったようですので，三重大学版も最新のものをベースに作り直しました。十分テストをしていませんので何かありましたらお教えください。

http://portal.mie-u.ac.jp/src/

評点平均: -
Haruhiko Okumura への返信

Re: セキュリティアップデート

- Tatsuya Shirai の投稿

　随分と一気に修正されましたね，Moodle1.9.1は特に．(Moodle1.9.2へ)

　function get_file_url()がとうとう本領発揮のようで，お陰でここしばらくは不都合が出るかも知れません．最終的にはＩＥ対応をここに集約できそうですが．

　ＨＴＭＬエディタ（htmlarea）も随分と修正されているようですが，何を目的としてどう変わったのか，よーく見極めないと危険ですね．

評点平均: -
Tatsuya Shirai への返信

Re: セキュリティアップデート

- Haruhiko Okumura の投稿
function get_file_url() の話は全然追えていません。^^;

きっとトラブルがあるんでしょうね。
http://oku.edu.mie-u.ac.jp/tex/
はもう1.9.2にしましたので，お試しください＞皆様。

評点平均: -
Haruhiko Okumura への返信

Re: セキュリティアップデート

- Tatsuya Shirai の投稿

　function get_file_url()は予想通り，当方の修正箇所と数箇所でバッティングしていました．取り敢えずは対処できましたし，好ましい修正です．さらに$typeを自分で増やせば，独自の処理を簡単に追加できるという意味でも面白い改善になっていました．

　それよりも今回の修正で追加された function send_temp_file()が気になります．まだこの関数を使用している箇所は一箇所だけですが，今後は利用が増えるのでしょうか．

lib/filelib.php，function send_temp_file()
 * Handles the sending of temporary file to user, download is forced.
 * File is deleted after abort or succesful sending.

successfulだろう，という突っ込みはおいておいて，send_file()関数が存在するのにもう一つ別の関数を増やすのは何故か．パッと見た感じではsend_file()関数よりは小機能で軽く，attachment（添付ファイル）限定，しかも送信したファイルは直ぐに削除されるというお手軽さから，サーバ側で作成したファイルをユーザ側にちょっと送り付けたい，という用途向けでしょうか．

　でも，send_file()では対応されているファイル名をダブルクオートで囲む改善が生かされていないのは勿体無いですね．

    @header('Content-Disposition: attachment; filename='.$filename);

評点平均: -