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

Unito
senooken ha unito 1 commit da tak4/main a gnusocialjp/main 1 anno fa
tak4 commented 1 anno fa

#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 anno fa
Proprietario

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

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

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

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

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

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

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

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

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

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

@tak4 修正が入ることで、相手からいいねの解除が来た場合の再送がなくなることを確認できました。 また、再投稿の場合は、二重化はそもそも発生していませんでした。OKです。ありがとうございました。
Questo contributo è stato incluso con successo!
Sign in to join this conversation.
Nessuna milestone
Nessun assegnatario
3 Partecipanti
Caricamento...
Annulla
Salva
Non ci sono ancora contenuti.