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

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1652 closed defect (fixed)

ファイルダウンロードの認証をトピック表示権限の確認に変更する

Reported by: tsukamoto Owned by: ebihara
Priority: minor Milestone: OpenPNE2.10.2
Component: その他 Version: 2.10.x & 2.11.x
Keywords: OpenPNE2.11.2 Cc:

Description (last modified by ebihara)

■現象

ファイルダウンロードの認証がトピックの表示権限があるかどうかでおこなわれていない。

公開コミュニティでは、非コミュニティメンバでもトピックを閲覧でき、添付ファイルへのリンクも表示される。

ダウンロード権限は、ダウンロードリンクを閲覧できる権限と合っているべきだと思う。また添付ファイルを含めてトピックコンテンツであることを考えれば、「リンクをコミュニティメンバ以外に表示しない」という解決より、「トピックを閲覧できる人は添付ファイルもダウンロードできる」という解決の方が自然だと思われる。

■原因

ファイルダウンロードの認証がコミュニティメンバーかどうかでおこなわれている。

■修正内容

webapp/pc/do/c_file_downloadで行っている認証を、コミュニティメンバか否かの判定から、トピックの表示権限の判定に変更する。

■関連情報

2.10.1ベースでのdiff

Index: webapp/modules/pc/do/c_file_download.php
===================================================================
--- webapp/modules/pc/do/c_file_download.php    (revision 224)
+++ webapp/modules/pc/do/c_file_download.php    (working copy)
@@ -29,7 +29,7 @@
         $c_commu_id = $c_topic['c_commu_id'];

         $status = db_common_commu_status($u, $c_commu_id);
-        if (!$status['is_commu_member']) {
+        if (!$status['is_bbs_view']) {
             handle_kengen_error();
         }

Change History (9)

comment:1 Changed 12 years ago by tejimaa

Milestone: OpenPNE2.11.2

私も、ご指摘の方が自然だと思います。 マイルストン変更しちゃいました。

実装してくれるかな。

comment:2 Changed 12 years ago by ogawa

Component: pne-frameworkその他
Milestone: OpenPNE2.11.2OpenPNE2.10.2
Priority: criticalminor
Version: 2.10.x2.10.x & 2.11.x

安定版に修正入れたいと思います。

comment:3 Changed 12 years ago by ebihara

Owner: changed from nobody to ebihara
Status: newassigned

やります

comment:4 Changed 12 years ago by ebihara

Keywords: 確認中 added

以下でパッチを取り込みました。ご確認ください。

comment:5 Changed 12 years ago by ogawa

Keywords: テスト待ち added; 確認中 removed

comment:6 Changed 12 years ago by kiwa

Resolution: fixed
Status: assignedclosed

安定版のみ確認しました。OKです。

comment:7 Changed 12 years ago by ebihara

Description: modified (diff)

comment:8 Changed 12 years ago by kiwa

Keywords: テスト待ち removed

comment:9 Changed 12 years ago by kiwa

Keywords: OpenPNE2.11.2 added
Note: See TracTickets for help on using tickets.