【提案】新しい連投規制 #56

kenmo-melon · 2023-10-19T23:27:40+09:00

kenmo-melon commented

2023-10-19 23:27:40 +09:00

responseテーブルにtimestampを追加
その人が過去40秒以内に行った書き込みをすべて検索
5秒以内の連投があった場合規制する

という感じです。以前のものはおそらくD1へのINSERTが反映されるのが少し遅いせいで効きが弱かったので、過去40秒を見ることにしました。40秒というのは勘で決めたのですが、30秒くらいでもいいかもしれません。
適当に試してみたのですが、クッキーを変えない単純なスクリプトであればほぼ確実に弾けます。3回くらいしか連投できないですね。もう一つ、INSERTを一箇所減らせるのもメリットかなと思います。
デメリットは、テーブルの変更が必要なことです。
めちゃくちゃ連投しているユーザーさんが書き込むと少し処理が増えますが、それでも高々長さ8のVecをソートするだけなのでまあ無視できる範囲かなと思います。

- responseテーブルにtimestampを追加 - その人が過去40秒以内に行った書き込みをすべて検索 - 5秒以内の連投があった場合規制するという感じです。以前のものはおそらくD1へのINSERTが反映されるのが少し遅いせいで効きが弱かったので、過去40秒を見ることにしました。40秒というのは勘で決めたのですが、30秒くらいでもいいかもしれません。適当に試してみたのですが、クッキーを変えない単純なスクリプトであればほぼ確実に弾けます。3回くらいしか連投できないですね。もう一つ、INSERTを一箇所減らせるのもメリットかなと思います。デメリットは、テーブルの変更が必要なことです。めちゃくちゃ連投しているユーザーさんが書き込むと少し処理が増えますが、それでも高々長さ8のVecをソートするだけなのでまあ無視できる範囲かなと思います。

kenmo-melon added 2 commits 2023-10-19 23:27:41 +09:00

cfe581ce38 Use the minimum res span in the last 40 minutes for limit res

107773cfe9 Avoid self.token_cookie.unwrap() if possible

kenmo-melon referenced this pull request

2023-10-19 23:35:02 +09:00

連投制限の改良？ #50

edginer commented

2023-10-20 00:21:55 +09:00

基本的には良さげで色々効きそうなんですが、試験的に導入して微妙にメトリクスとの相談が必要かもしれません

edginer reviewed 2023-10-20 00:23:23 +09:00

src/migrations/add-timestamp-to-res_2023-10-19/up.sql

													
				@ -0,0 +1,2 @@

				ALTER TABLE authed_cookies DROP COLUMN last_wrote_time;

				ALTER TABLE responses ADD COLUMN timestamp INTEGER default 0;