Project

General

Profile

Bug(バグ) #2769

つぶやきAPIに連投禁止機能

Added by 守 手嶋 over 11 years ago. Updated about 3 years ago.

Status:
Rejected(差し戻し)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2012-01-27
Due date:
% Done:

50%

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

Description

一つ前の投稿と比較して、全く同じ内容の場合は、ブロックする機能。
TwitterAPIには付いている。


Subtasks

Backport(バックポート) #4398: つぶやきAPIに連投禁止機能New(新規)isao sano

Backport(バックポート) #4399: つぶやきAPIに連投禁止機能New(新規)isao sano


Related issues

Copied to OpenPNE 3 - Backport(バックポート) #4398: つぶやきAPIに連投禁止機能 New(新規) 2012-01-27

History

#1 Updated by 守 手嶋 over 11 years ago

  • Project changed from HOUOU to OpenPNE 3
  • Target version deleted (HOUOU_JANUARY)

#2 Updated by 守 手嶋 over 11 years ago

  • Target version set to 253

#3 Updated by Shouta Kashiwagi over 11 years ago

  • Target version changed from 253 to OpenPNE 3.8.x

#4 Updated by kaoru n almost 11 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to kaoru n
  • 3.8 で発生するか set to Unknown (未調査)

#5 Updated by kaoru n almost 11 years ago

  • 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)
  • 3.8 で発生するか changed from Unknown (未調査) to Yes (はい)

3.8はスマートフォンAPI使用時にも発生します。

#6 Updated by kaoru n almost 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • Assignee deleted (kaoru n)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/59
プルリクエストしました。

#7 Updated by Shinichi Urabe over 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

気になる点

  • 例えばあまりにも早い連投で DB が Master-Slave 環境あれば、参照用 Slave が少し遅延した場合、連投と見なされない可能性がある
  • スクリプトを使えば、少しずつ加工した文字列の連投が可能

この修正はバグとして扱うべきものとは個人的には思えない
用途にもよるが、OpenPNE と Twitter の用途は同じでない場合もあるのではないか

強制的に連投禁止するのではなく、設定があってもよいかと思う

  • 普通の利用者でも同じ文字列の投稿はあり得る
      • A げんきですか
      • B はい
      • A いい天気ですね
      • B はい

仕様について議論された上でソースコードのレビューを行う形がよいかと考える

#8 Updated by isao sano over 3 years ago

#9 Updated by kaoru n about 3 years ago

  • Target version changed from OpenPNE 3.8.x to OpenPNE 3.11.x

Also available in: Atom PDF