Project

General

Profile

Bug(バグ) #1558

[api] RESTful API's standard query parameters の count について上限を設定できるようにする

Added by Shogo Kawahara almost 9 years ago. Updated almost 9 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-09-06
Due date:
% Done:

100%

3.6 で発生するか:
[QA]バグ通知済:
No
3.8 で発生するか:
Unknown (未調査)

Description

Overview

現状、count=1000 などで指定するとサーバに大きな負荷となる可能性がある。

refs: standard query parameters

http://www.opensocial.org/Technical-Resources/opensocial-spec-v09/REST-API.html#rfc.section.7.8

Way to fix

  • 上限値を sfConfig::get('op_opensocial_api_max_count', 100); などで設定する
    • 上限値以上をcountに指定した場合、countは上限値に設定されるようにする。
  • 関係箇所
    • lib/util/opJsonDbOpensocialService.class.php
      • L75
      • L167
      • L563
      • L659

メモ:

  • 件数取得用メソッドを、opJsonDbOpensocialServiceに作成するべきかもしれない。
  • op_opensocial_api_default_count とかにデフォルトの件数を設定できるようにするべきかもしれない。

Environment

opOpenSocialPlugin0.9.x〜


Related issues

Related to opOpenSocialPlugin - Backport(バックポート) #1717: [api] RESTful API's standard query parameters の count について上限を設定できるようにする Fixed(完了) 2010-09-06
Related to opOpenSocialPlugin - Backport(バックポート) #1722: [api] RESTful API's standard query parameters の count について上限を設定できるようにする Fixed(完了) 2010-09-06
Related to opOpenSocialPlugin - Backport(バックポート) #1723: [api] RESTful API's standard query parameters の count について上限を設定できるようにする Fixed(完了) 2010-09-06

Associated revisions

Revision f54235f5 (diff)
Added by Shogo Kawahara almost 9 years ago

fixed opJsonDbOpensocialService to limit count of API results (fixes #1558)

History

#1 Updated by Shogo Kawahara almost 9 years ago

  • Assignee set to Shogo Kawahara

#2 Updated by Anonymous almost 9 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#3 Updated by Anonymous almost 9 years ago

更新履歴 commit:"8fa92ec8f38c8f4d3df1665343051f5266e46188" で適用されました。

#4 Updated by Shogo Kawahara almost 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Fixed(完了)
  • % Done changed from 50 to 100

Also available in: Atom PDF