プロジェクト

全般

プロフィール

Bug(バグ) #3273

activity/search.json?member_id=? で特定メンバーのアクティビティを取得すると500エラーとなる

Youichi Kimura11年以上前に追加. ほぼ5年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2012-12-06
期日:
進捗率:

100%

3.6 で発生するか:
No (いいえ)
3.8 で発生するか:
Yes (はい)

説明

現象

MySQL を使用している SNS で、API を使用して http://sns.example.com/api.php/activity/search.json?member_id=1 のように特定メンバーのアクティビティを取得しようとすると Internal Server Error が返ってしまう場合がある。

原因

opActivityQueryBuilder::buildMemberQuery() 内で andWhereIn メソッドを使用している。ここで生成されるクエリはサブクエリとして使用されるため #3135 で報告されているように不正なクエリが生成されてしまう。

  protected function buildMemberQuery($query, $memberId = null, $publicFlag = ActivityDataTable::PUBLIC_FLAG_SNS)
  {
    if (is_array($memberId))
    {   
      $query->andWhereIn('a.member_id', $memberId);
    }   
    elseif ($memberId instanceof Doctrine_Query)
    {   
      $query->andWhere('a.member_id IN ('.$memberId->getDql().')');
    }   
    elseif (is_scalar($memberId))
    {   
      $query->andWhere('a.member_id = ?', $memberId);
    }   

    $query->andWhereIn('a.public_flag', $this->table->getViewablePublicFlags($publicFlag));

    return $query;
  }

修正内容

opActivityQueryBuilder::buildMemberQuery() メソッドに #3135 の問題を回避する修正を施す


子チケット

Backport(バックポート) #3331: activity/search.json?member_id=? で特定メンバーのアクティビティを取得すると500エラーとなるFixed(完了)kaoru n


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #3135: サブクエリのWHERE節に「field IN ?」の形式のDQLがあると Invalid parameter number エラーが発生する New(新規) 2012-07-27
関連している OpenPNE 3 - Bug(バグ) #3344: opActivityQueryBuilder で特定のメソッドで配列を引き数に渡した場合にエラーが発生する New(新規) 2013-05-07
関連している OpenPNE 3 - Bug(バグ) #4260: opTimelinePlugin 携帯版プロフィール画面のSNSメンバーのタイムラインガジェット Web公開していない投稿がWeb公開されてしまう Fixed(完了) 2017-11-15

関係しているリビジョン

リビジョン af729c35 (差分)
Youichi Kimura約11年前に追加

avoid invalid where...in parameter bug (fixes #3273)

リビジョン 54e27511 (差分)
kaoru nほぼ5年前に追加

(refs #3273) public_flag の数によって addWhere または addWhereIn の使用を分岐するよう修正

リビジョン 9d778779 (差分)
kaoru nほぼ5年前に追加

(refs #3273) 定義した変数を使用するよう修正

リビジョン f2937158
kaoru nほぼ5年前に追加

Merge pull request #520 from nishizoe/t-3273

(refs #3273) public_flag の数によって addWhere または addWhereIn の使用を分岐するよう修正

履歴

#1 Youichi Kimura11年以上前に更新

  • 説明 を更新 (diff)

#2 Youichi Kimura11年以上前に更新

  • 説明 を更新 (diff)

#3 Youichi Kimura約11年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 af729c356aa54780e6a2a2e4870efd1db21a42f3 で適用されました。

#4 Chiharu Nakajimaほぼ11年前に更新

  • 3.6 で発生するかUnknown (未調査) から No (いいえ) に変更
  • 3.8 で発生するかUnknown (未調査) から Yes (はい) に変更

#5 Yuya Watanabeほぼ11年前に更新

https://redmine.openpne.jp/issues/3135#note-1 こちらに原因を書きましたが, #3135 の修正を適用した場合本チケットの修正が不要になり, Invalid とする可能性があります.

#6 Yuya Watanabeほぼ11年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

#3135 の修正が難しそうなのでこちらのチケットに対応している修正をレビューしました.

確かにオープンソースの OpenPNE の実装では大小関係による比較によって公開範囲の区別はできます.
ただ,この公開範囲については公開範囲の追加のようなカスタマイズを行う場合が考えられます.
このようなカスタマイズを行う場合,簡単に考えられるデータ定義として公開範囲の定数を増やすことになりますが,増やす場合に定数の数字を大きくしたものを用いるでしょう.
単純に大小関係による比較という実装を行い,この修正をカスタマイズした OpenPNE に適用した場合に「非公開」設定のものがクエリの抽出対象に含まれ,意図していない公開が発生しうるため危険な修正であると思いました.

動作しないということが公開範囲よりも問題である場合や公開範囲についてカスタマイズをしていないという SNS には今の修正は有効だと思いますが,上記理由により差し戻しとしておきます.

#7 Yuya Watanabeほぼ11年前に更新

このチケットを対応している間に気になった点について別チケット #3344 に記述しました.

#8 isao sanoほぼ7年前に更新

対象バージョン変更のため、修正内容の確認を行います。

#9 isao sanoほぼ7年前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

#10 kaoru nほぼ5年前に更新

#3273-3 は、old-master にのみ適用済み(2019/04/12 現在)

#11 kaoru nほぼ5年前に更新

  • 関連している Bug(バグ) #4260: opTimelinePlugin 携帯版プロフィール画面のSNSメンバーのタイムラインガジェット Web公開していない投稿がWeb公開されてしまう を追加

#12 kaoru nほぼ5年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 担当者Youichi Kimura から kaoru n に変更
  • 進捗率50 から 0 に変更

表題の件について、

    $query->andWhereIn('a.public_flag', $this->table->getViewablePublicFlags($publicFlag));
の部分において、
「$this->table->getViewablePublicFlags($publicFlag)」のカウントが「1」である場合は、
addWhereIn ではなく、addWhere を使用するように修正します。
問題の回避のための修正です。

#13 kaoru nほぼ5年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

https://github.com/openpne/OpenPNE3/pull/520
にてプルリクエストを行いました

#14 Rimpei Ogawaほぼ5年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

#15 kaoru nほぼ5年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

#16 Rimpei Ogawaほぼ5年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

レビューOKです。

#17 kaoru nほぼ5年前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE-3.9.5 に変更

#18 isao sanoほぼ5年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

試験完了しました。
問題ありません。

#19 kaoru nほぼ5年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF