ブロック時においてフォローリクエストのキャンセル処理で一件のみ処理される

[ikasoba]

💡 Summary

ブロック時にフォローリクエストのキャンセル処理が行われるようですが複数を考慮しておらず、一件のみ削除される実装となっているようです。

misskey/packages/backend/src/core/UserBlockingService.ts

Lines 61 to 66 in 19384ef

	public async block(blocker: MiUser, blockee: MiUser, silent = false) {
	await Promise.all([
	this.cancelRequest(blocker, blockee, silent),
	this.cancelRequest(blockee, blocker, silent),
	this.userFollowingService.unfollow(blocker, blockee, silent),
	this.userFollowingService.unfollow(blockee, blocker, silent),

misskey/packages/backend/src/core/UserBlockingService.ts

Lines 95 to 102 in 19384ef

	private async cancelRequest(follower: MiUser, followee: MiUser, silent = false) {
	const request = await this.followRequestsRepository.findOneBy({
	followeeId: followee.id,
	followerId: follower.id,
	});
	if (request == null) {
	return;

発信側の実装などの都合によっては適切に解釈してもらえない可能性もあると思われます。

🥰 Expected Behavior

• 複数件を考慮した実装

🤬 Actual Behavior

• 一件のみキャンセルされる

📝 Steps to Reproduce

No response

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey:

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey: 
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

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