Develop に追従するとログインできない(dc3629e)

[kanarikanaru]

💡 Summary

feat(backend): report `Retry-After` if client hit rate limit (#13949) · misskey-dev/misskey@dc3629e · GitHub
これを適用すると、新規にログインできない等の問題が発生する。
現状Developブランチにのみにあるコミットです。

🥰 Expected Behavior

• 新規にログインできる
• デフォルトロールでノートやリアクションが消せる

🤬 Actual Behavior

• ログインするためにPOSTしても結果が返ってこない
• nginxのリバースプロキシを経由させなくても発生

📝 Steps to Reproduce

1. Developに追従する
2. 新しいブラウザでログインを試みる
3. ぐるぐる

💻 Frontend Environment

* Model and OS of the device(s): MacBook Pro (14inch, 2021), macOS Sonoma 14.5

* Browser: Chrome 125
* Server URL:misskey.backspace.fm
* Misskey:2024.05(Developに追従、該当コミットをrevertして通常通り動作することを確認)

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:systemd
* Misskey: 2024.05(Developに追従、該当コミットをrevertして通常通り動作することを確認)
* Node: 20.14
* PostgreSQL: 15
* Redis: 7
* OS and Architecture: Ubuntu 22

Do you want to address this bug yourself?

• Yes, I will patch the bug myself and send a pull request

[tai-cha]

追加：新規アカウント作成も通らないかも
→ とても長いローディングのあとデータのみ作成された（アカウントのチュートリアルに進まないもののユーザーは作成された）

[tai-cha]

ローカルdev環境では再現できず、misskey-tgaでは再現できる

[u1-liquid]

ローカルdev環境では再現できず

👀

misskey/packages/backend/src/server/api/RateLimiterService.ts

Lines 28 to 30 in c51347d

	if (process.env.NODE_ENV !== 'production') {
	this.disabled = true;
	}

[zyoshoka]

misskey/packages/backend/src/server/api/RateLimiterService.ts

Line 60 in c51347d

return max;

記法的に良いのか分かりませんがここをとりあえず

return max.then(ok).catch(e => reject(e));

とすると直りました

[anatawa12]

その方針で行くのであればmax.then(ok, reject)のほうが良さそう。PRはやします

[tai-cha]

自分も
return max.then(ok).catch(reject);
で動くのは確認してPR出そうと思ってました（anatawa12さんのでよさそう）

[anatawa12]

ok(Promiseのresolveコールバック)がエラー出すこと多分ないけどthen(ok).catch(reject)だと両方呼ばれるリスクがあるので一応thenに２つのほうが良さそうとしました

[KisaragiEffective]

off-topic(?): resolveを呼んでいないのはESLintのno-floating-promiseで検知されているかと思ったが、CI実行時に--quietがついているためおそらく抑制されている。これも帰ったら見る。

[KisaragiEffective]

off-topic(?): resolveを呼んでいないのはESLintのno-floating-promiseで検知されているかと思ったが、CI実行時に--quietがついているためおそらく抑制されている。これも帰ったら見る。

→ c51347d において /misskey/packages/backend $ eslint "src/*/**.ts" を実行してもno-floating-promiseが発火されていない。

[anatawa12]

voidにreturnしてるところで発火してもいい気はするけど、new Promiseのresolveに関してはlimiter.getのコールバックがいつどのように呼ばれるか不定だからlinterがどうにかできる範疇を超えてるきはする