#17 「いいね解除」された時に投稿が増えるバグ、及び、付随するバグの修正

Merged
senooken merged 1 commits from tak4/main into gnusocialjp/main 1 year ago
tak4 commented 1 year ago

#10 の残りのバグを修正しました。
mastodonとの会話で増幅するバグについては、再現方法が分らないので未確認ですが、今回の修正で直っている可能性があります。
また、リモートとの「いいね解除」の通信がうまく行かない事があったので、そちらも修正しました。

  • 外部サーバーから「いいね解除」された時の増幅を防止する。
    Activitypub_notice.php
  • 「いいね解除」した事を外部サーバーに通知する。
    Activitypub_like.php
  • 外部サーバーからの「いいね解除」をローカルDBに反映する。
    Activitypub_like.php

( #14 の反映はできていないので、もし #14 を先にマージされるのであれば、その後で修正後、再度プルリクエストします。)

#10 の残りのバグを修正しました。 mastodonとの会話で増幅するバグについては、再現方法が分らないので未確認ですが、今回の修正で直っている可能性があります。 また、リモートとの「いいね解除」の通信がうまく行かない事があったので、そちらも修正しました。 - 外部サーバーから「いいね解除」された時の増幅を防止する。 Activitypub_notice.php - 「いいね解除」した事を外部サーバーに通知する。 Activitypub_like.php - 外部サーバーからの「いいね解除」をローカルDBに反映する。 Activitypub_like.php ( #14 の反映はできていないので、もし #14 を先にマージされるのであれば、その後で修正後、再度プルリクエストします。)
senooken commented 1 year ago
Owner

対応ありがとうございます!内容、修正量的に、こちらを先にマージしたいです。PHP 8対応は確認や修正に時間がかかると思います。 他の作業との兼ね合いで、確認に時間がかかるかもしれませんので、お待ちください…

対応ありがとうございます!内容、修正量的に、こちらを先にマージしたいです。PHP 8対応は確認や修正に時間がかかると思います。 他の作業との兼ね合いで、確認に時間がかかるかもしれませんので、お待ちください…
tak4 commented 1 year ago
Poster

承知いたしました。よろしくお願いいたします。

承知いたしました。よろしくお願いいたします。

tak4さんの変更は、PHP8対応をマージ前にも、問題無しでこれをマージ出来るはずです

tak4さんの変更は、PHP8対応をマージ前にも、問題無しでこれをマージ出来るはずです
senooken commented 1 year ago
Owner

suwako、コメントありがとうございます。ピンポイントの修正なのでおそらく大丈夫だろうと私も思っています。現状この2個しかマージ予定ないので、修正量の少ないこちらをやはり先にマージします。

suwako、コメントありがとうございます。ピンポイントの修正なのでおそらく大丈夫だろうと私も思っています。現状この2個しかマージ予定ないので、修正量の少ないこちらをやはり先にマージします。
senooken commented 1 year ago
Owner

@tak4 修正が入ることで、相手からいいねの解除が来た場合の再送がなくなることを確認できました。

また、再投稿の場合は、二重化はそもそも発生していませんでした。OKです。ありがとうございました。

@tak4 修正が入ることで、相手からいいねの解除が来た場合の再送がなくなることを確認できました。 また、再投稿の場合は、二重化はそもそも発生していませんでした。OKです。ありがとうございました。
This pull request has been merged successfully!
Sign in to join this conversation.
No Milestone
No assignee
3 Participants
Loading...
Cancel
Save
There is no content yet.