4drian3d / 4drian3d/SignedVelocity

Issue warning, if possible, when behavior changes due to backend plugin handling

オープン
#17 コメント 4 件 リアクション 2 件 担当者 1 名 @4drian3d が担当を希望しています GitHub で見る
bug
主要言語
Java
スター
73
フォーク
23
平均マージ
11時間 19分
マージ済み PR(30日)
9

説明

In the pre-existing Velocity API, canceling a chat event prevents it from being sent to the backend servers. Chat canceled on the proxy is therefore invisible to Bukkit plugins. However, using SignedVelocity fundamentally alters this behavior by forwarding **all** chat events, even if canceled, to the backend servers. While this makes the plugin work, it is a behavioral change from the prior status quo, and leaks the abstraction SignedVelocity employs to shore up chat handling in a post-1.19.4 world.

This can be confusing for users who are acquainted to the previous behavior. For example, it may cause certain proxy plugin features to stop working when switching from pre-1.19.4 to post-1.19.4 with SignedVelocity, since the behavior of chat events is now different.

Most of the time, however, uncanceling a canceled chat event doesn't make sense. Fortunately, SignedVelocity is in the unique position of knowing exactly when a chat event was canceled by the proxy but uncanceled by the backend server. I therefore propose SignedVelocity add another listener on `EventPriority.HIGHEST` to log a brief warning if behavior has fundamentally changed as a result of a backend plugin *uncanceling* the event. It should be relatively straightforward given you already have a queue of chat data, so no additional data structure overhead would be incurred. If you want to avoid annoying users, the brief warning can be one-time, or it can be disabled by a config option. However, I argue it should be enabled by default, because it presents a behavioral change from the typical Velocity API proxy plugins are used to.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

Look at the chat event handling code in SignedVelocity, likely where chat events are queued and forwarded. Identify the listener that currently uncancels events for backend servers. Add a HIGHEST priority listener to detect when a backend plugin uncancels a proxy-canceled event. Implement a configurable warning log, possibly one-time per event type. Check existing configuration options for adding a new toggle.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java
領域
backend, devtools
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。