meron
meron
> アンテナのノートを取得したときに再度有効化される 私の理解が間違ってなければですがこれはおそらくこの意図通りには動いてなくて 受信されたノートは `NoteCreateService.ts`の`create()`->`postNoteCreated()`->`AntennaService.ts`の`addNoteToAntennas()`->... と流されていくと思うのですが https://github.com/misskey-dev/misskey/blob/develop/packages/backend/src/core/AntennaService.ts#LL90C3-L90C3 上記で呼び出してる`const antennas = await this.getAntennas();`で帰ってくるのは https://github.com/misskey-dev/misskey/blob/develop/packages/backend/src/core/AntennaService.ts#L187 上記の通り`isActive: true`のものだけとなっており無効となっているアンテナにはノートが送られない、つまり`antennas/notes.ts`がこの後呼ばれることはないと思っております。
あー!!!今ものすごい根本的な勘違いに気が付いたantennas/notesが叩かれてないじゃなくて、 update()が走る前に空配列を返す条件に引っかかって有効にするロジックに入ってないが正解かも https://github.com/misskey-dev/misskey/blob/develop/packages/backend/src/server/api/endpoints/antennas/notes.ts#L86-L94
アンテナが0件になってしまった後の再有効化をどこでやるのが相応しそうか考えてたのですが、 0件になってしまうということはアンテナの条件に問題があることが多そう(0件で動いてなさそうならユーザーは自然と設定項目を再度弄るという予想)ということでこのプルリクのantennas/updateの変更だけにしてantennas/showの方は元に戻そうと考えていますがどうでしょうか?
利用者側としては以下のように通知欄に残り続けてしまう為、モデレーターによって適切に対処されたのか分かりにくいのもあります。(マウスオーバーしたりクリックすると永遠にロード中になったり、エラー画面になったりはしますが) ただ他の削除関連のIssuesとかもあるので単純に物理削除すればOK、とはいかないかもですが
> 25msが短すぎた疑惑はある 確かに、事象からロードまで流れて、ほかのサウンド再生要求があった際既に`canPlay == true`なのかなと思いましたけどPromiseが完了(又はエラー)になるまでfinally()が呼ばれないのであれば単純に短すぎて効果が無かった説もありそうですね。
このIssue書いてからも何度か発生していますがリロード時だけでなくバックグラウンドに追いやってしばらくしてから再度フォアグラウンドに戻したら爆音になったり、ボーっとTLを眺めているときにも極まれに爆音となったりと音が重なってはなさそうな状態でも発生したりするので絞り込めるどころか謎が深まってきてしまった・・・
> Firefoxで起こったものであるならば、ブラウザ側のバグの可能性がある 自環境は以下なのでブラウザで起因はなさそうかもです。 Windows 11 Google Chrome 120.0.6099.200(Official Build) (64 ビット)
`DISABLE_AUTOMATIC_SWITCHING_TO_APPROVED_REGISTRATIONS`を設定して放置はあり得ると思うので無効になるのではなく無効になる期間が伸ばせるだけ(1weekが3weekあたりに伸びるだけ)とかに抑えた方がよさそうな気がする。 どうしても止めたいなら責任もってソース改変してでもいい気がしますし