anvilpowered / catalyst Goto Github PK

Basically the title. An ability to search on discord by in-game username would be cool.

Describe the bug
A clear description of the bug:
Inside the config where you set the game status for the discord bot you can not use the %players% placeholder to pull the player count if you wanted to show it in the game status section of the bot.
To Reproduce
Steps to reproduce the behavior:
Try to put %players% in the "playing" field
Expected behavior
A clear and concise description of what you expected to happen.
%players% being replaced with the player count of the network.
Screenshots
If applicable, add screenshots to help explain your problem.
https://gyazo.com/dd0367c3938cf52a29ba144fbde7cead

Desktop (please complete the following information):

• Velocity Version: 1.1.0
• Catalyst Version: 0.1.3
• Additional Plugins installed on velocity:

Additional context
Add any other context about the problem here.

A clear description of the feature you'd like to see implemented

Allow commands sent in discord to be executed in console. Create a config option for an allowed role or allowed members and register slash commands to allow punishment and other commands to be executed from discord.

Hi im running a multi velocity server. currently all the servers are running on the same build of spigot for 1.16.2

When running the /server command, im met with "the server you are attempting to connect to is running a different Minecraft version!" all the servers should be on the EXACT same build. We dont run ViaVersion and have tried turning the version check off via the config but we still get this error.

A clear description of the feature you'd like to see implemented

In order to allow servers to use ViaVersion, there has to be a change implemented into the ServerCommand. As it stands, the ServerCommand checks if the version of the targeted server is the same version as the player that is attempting to connect, however if the backend servers are running ViaVersion, this will block them from being able to connect to other servers on their proxy.
To allow this to occur, there will need to be further abstraction of the ServerCommand, and also an addition to the config.

A clear description of the feature you'd like to see implemented

Support @a operator in /send command

A clear description of the feature you'd like to see implemented

Be able to customise how names are displayed in the tablist specifically the spacing and such and remove things like Prefix and also if possible implement an option to add "is AFK" at the end of someone's name if they are AFK.

[09:51:54 INFO] [catalyst]: xPaz has joined the proxy
[09:51:54 ERROR]: Oct 29, 2020 9:51:54 AM com.google.common.eventbus.EventBus$LoggingHandler handleException
[09:51:54 ERROR]: SEVERE: Exception thrown by subscriber method onPlayerJoinEvent(org.anvilpowered.catalyst.api.event.JoinEvent) on subscriber org.anvilpowered.catalyst.common.listener.CommonDiscordChatListener@2c806135 when dispatching event: org.anvilpowered.catalyst.common.event.CommonJoinEvent@15b6b633
[09:51:54 ERROR]: java.lang.NullPointerException
[09:51:54 ERROR]: at org.anvilpowered.catalyst.common.discord.CommonWebhookSender.getWebhook(CommonWebhookSender.java:81)
[09:51:54 ERROR]: at org.anvilpowered.catalyst.common.discord.CommonWebhookSender.sendWebhookMessage(CommonWebhookSender.java:49)
[09:51:54 ERROR]: at org.anvilpowered.catalyst.common.listener.CommonDiscordChatListener.onPlayerJoinEvent(CommonDiscordChatListener.java:156)
[09:51:54 ERROR]: at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[09:51:54 ERROR]: at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
[09:51:54 ERROR]: at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[09:51:54 ERROR]: at java.lang.reflect.Method.invoke(Method.java:498)
[09:51:54 ERROR]: at com.google.common.eventbus.Subscriber.invokeSubscriberMethod(Subscriber.java:87)
[09:51:54 ERROR]: at com.google.common.eventbus.Subscriber$SynchronizedSubscriber.invokeSubscriberMethod(Subscriber.java:144)
[09:51:54 ERROR]: at com.google.common.eventbus.Subscriber$1.run(Subscriber.java:72)
[09:51:54 ERROR]: at com.google.common.util.concurrent.DirectExecutor.execute(DirectExecutor.java:30)
[09:51:54 ERROR]: at com.google.common.eventbus.Subscriber.dispatchEvent(Subscriber.java:67)
[09:51:54 ERROR]: at com.google.common.eventbus.Dispatcher$PerThreadQueuedDispatcher.dispatch(Dispatcher.java:108)
[09:51:54 ERROR]: at com.google.common.eventbus.EventBus.post(EventBus.java:212)
[09:51:54 ERROR]: at org.anvilpowered.catalyst.velocity.listener.VelocityListener.lambda$null$1(VelocityListener.java:139)
[09:51:54 ERROR]: at java.util.concurrent.CompletableFuture.uniAccept(CompletableFuture.java:670)
[09:51:54 ERROR]: at java.util.concurrent.CompletableFuture$UniAccept.tryFire(CompletableFuture.java:646)
[09:51:54 ERROR]: at java.util.concurrent.CompletableFuture$Completion.exec(CompletableFuture.java:457)
[09:51:54 ERROR]: at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
[09:51:54 ERROR]: at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
[09:51:54 ERROR]: at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
[09:51:54 ERROR]: at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:163)
[09:51:54 ERROR]:

[09:56:02 INFO] [catalyst]: global : [!]* xPaz : fasfasdfdff
[09:56:02 ERROR]: Oct 29, 2020 9:56:02 AM com.google.common.eventbus.EventBus$LoggingHandler handleException
[09:56:02 ERROR]: SEVERE: Exception thrown by subscriber method onChatEvent(org.anvilpowered.catalyst.api.event.ChatEvent) on subscriber org.anvilpowered.catalyst.common.listener.CommonDiscordChatListener@2c806135 when dispatching event: org.anvilpowered.catalyst.common.event.CommonChatEvent@5a240bef
[09:56:02 ERROR]: java.lang.NullPointerException
[09:56:02 ERROR]: at org.anvilpowered.catalyst.common.discord.CommonWebhookSender.getWebhook(CommonWebhookSender.java:81)
[09:56:02 ERROR]: at org.anvilpowered.catalyst.common.discord.CommonWebhookSender.sendWebhookMessage(CommonWebhookSender.java:49)
[09:56:02 ERROR]: at org.anvilpowered.catalyst.common.listener.CommonDiscordChatListener.onChatEvent(CommonDiscordChatListener.java:94)
[09:56:02 ERROR]: at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[09:56:02 ERROR]: at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
[09:56:02 ERROR]: at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[09:56:02 ERROR]: at java.lang.reflect.Method.invoke(Method.java:498)
[09:56:02 ERROR]: at com.google.common.eventbus.Subscriber.invokeSubscriberMethod(Subscriber.java:87)
[09:56:02 ERROR]: at com.google.common.eventbus.Subscriber$SynchronizedSubscriber.invokeSubscriberMethod(Subscriber.java:144)
[09:56:02 ERROR]: at com.google.common.eventbus.Subscriber$1.run(Subscriber.java:72)
[09:56:02 ERROR]: at com.google.common.util.concurrent.DirectExecutor.execute(DirectExecutor.java:30)
[09:56:02 ERROR]: at com.google.common.eventbus.Subscriber.dispatchEvent(Subscriber.java:67)
[09:56:02 ERROR]: at com.google.common.eventbus.Dispatcher$PerThreadQueuedDispatcher.dispatch(Dispatcher.java:108)
[09:56:02 ERROR]: at com.google.common.eventbus.EventBus.post(EventBus.java:212)
[09:56:02 ERROR]: at org.anvilpowered.catalyst.velocity.listener.VelocityListener.lambda$onChat$4(VelocityListener.java:198)
[09:56:02 ERROR]: at java.util.concurrent.CompletableFuture.uniAccept(CompletableFuture.java:670)
[09:56:02 ERROR]: at java.util.concurrent.CompletableFuture$UniAccept.tryFire(CompletableFuture.java:646)
[09:56:02 ERROR]: at java.util.concurrent.CompletableFuture$Completion.exec(CompletableFuture.java:457)
[09:56:02 ERROR]: at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
[09:56:02 ERROR]: at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
[09:56:02 ERROR]: at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
[09:56:02 ERROR]: at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:163)
[09:56:02 ERROR]:

Desktop (please complete the following information):

• Velocity Version: Velocity 1.1.2-SNAPSHOT (git-f3e1a7e0-b294)
• Catalyst Version: 08b6885 / build #60 on CI, also happens on 0.2.1 (both using matching anvil versions)
• Additional Plugins installed on velocity: anvil, bungeequack, luckperms, catalyst, craftingstore, joinkick, minimotd-velocity, pingcommand, plan, velocityguard

Describe the bug
A clear description of the bug: No idea

To Reproduce
Steps to reproduce the behavior: Just sorta happened

Expected behavior
A clear and concise description of what you expected to happen.
No bug to spam my console and not make my eyes hurt

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

• Velocity Version: Velocity 1.1.8
• Catalyst Version: 0.3.2
• Additional Plugins installed on velocity: anvil, luckperms, skinsrestorer, tcpshield, vbans

Additional context
Add any other context about the problem here.
https://hastebin.com/isanivohuf.properties
is the link to the error

A clear description of the feature you'd like to see implemented
When a message is sent in discord it is not checked for blacklisted words before it is sent through to minecraft. Should be pretty easy to implement.

Describe the bug
A clear description of the bug:
When using ServerListPlus to run the MOTD and player count message you get a large error message whenever pining the server (even if invalid hostname)

To Reproduce
Steps to reproduce the behaviour:

1. Install ServerListPlus dev build for Velocity
2. Install Catalyst and Anvil
3. Launch Velocity
4. Refresh multiplayer list in Minecraft so server is pinged
5. profit?

Expected behavior
A clear and concise description of what you expected to happen.
Not to have the giant error message.

Screenshots
If applicable, add screenshots to help explain your problem.
see log below

Desktop (please complete the following information):

• Velocity Version: Velocity 1.1.0-SNAPSHOT (git-5d287306-b243)
• Catalyst Version: 0.2.1
• Additional Plugins installed on velocity: LuckPerms, Maintenance, ServerListPlus.

Additional context
Add any other context about the problem here.
https://pastebin.com/raw/X4DLdpGE

Some plugins require the user to type something in chat, and this doesn't work while using proxy based chat (globla chat).
If you forwarded the chat message to the server as well, this can be avoided. Maybe a helper plugin to stop duplicate messages appearing. I don't know.

Describe the bug
A clear description of the bug:
When a user types @everyone or @here in Minecraft chat once this is streamed into discord it actually permits sending of the notification to ping all users.

To Reproduce
Steps to reproduce the behavior:

1. Setup catalyst with discord integration
2. type @everyone or @here
3. profit?

Expected behavior
A clear and concise description of what you expected to happen.
Not to recognise the command or not act on it or post it with it being quoted out.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

• Velocity Version: Velocity 1.1.0-SNAPSHOT.
• Catalyst Version: 0.2.1
• Additional Plugins installed on velocity: n/a

Describe the bug
TAB does not work

To Reproduce
Use latest stable release of velocity

Expected behavior
TAB works like b4

Screenshots
After:

Desktop (please complete the following information):

• Velocity Version: 3.0
• Catalyst Version: 0.3
• Additional Plugins installed on velocity: LuckPerms, Nuvotifier, VotingPlugin, Snap, Plan and my private plugin that just adds +1 to slots

Add a command for discord so you can see who is online on servers.
!online / !players

Describe the bug
A clear description of the bug:
When I type in chat message is not does not show for me or any other player
stafchat is visible only to person that types message other staff don't see messages

To Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

• Velocity Version: 1.1.0-snapshoot
• Catalyst Version: 0.2.0
• Additional Plugins installed on velocity:
  

Log:
latest(1).log

The default behavior of velocity adds the /server <servername> command where the <servername> can be tab completed.

Request: Add a toggle to either

• Disable the /server (without an argument) command and re-enable the default behavior.
• Enable the /server <servername> tab-complete in addition to the /server (without an argument).

Describe the bug
When enable-status in server.properties is false, Failed to connect is displayed.

To Reproduce

1. Set up Velocity & Catalyst.
2. Set enable-status in server.properties to false.
3. Failed to connect.

Expected behavior
The player will be successfully connected to the backend server.

Screenshots

Desktop (please complete the following information):

• Velocity Version: 1.1.5
• Catalyst Version: 0.3.2
• Additional Plugins installed on velocity: Anvil 0.3.0

Additional context
This issue does not occur with Velocity alone, so you may want to mark it as a bug. If this is not fixed, it might be a good idea to provide a FAQ.

This issue is a bit tricky for the following reasons:

• No errors will be output to the console at all.
• Console of the backend server will log as if the player connected for a moment and then immediately disconnected.

Almost certainly, the following checks are causing this issue.
https://github.com/AnvilPowered/Catalyst/blob/088793a6de30fe200169ef365a1a8a319c127492/catalyst-velocity/src/main/java/org/anvilpowered/catalyst/velocity/listener/VelocityListener.java#L119-L120

However, simply removing this check does not seem like a good idea. I don't understand Java and Velocity, so I can't come up with a good solution.

A clear description of the feature you'd like to see implemented
Using LuckPerms weight system have the Tablist automatically order it highest to lowest.

Example:
Owner - weight 10
Admin weight 9
Mod weight 8
VIP Weight 5
Default weight 0

Would display in that order say there is 6 players online

Bob - Admin
Dave - Owner
Sam - Mod
Jade - VIP
Cableguy20 - default

These would then display on the tablist like so

• (Owner) Dave
• (Admin) Bob
• (Mod) Sam
• (VIP) Jade
• (Member) Cableguy20.

I commented on this on the velocity forums, but the configuration leaves much to be desired.
eg: disabling certain features, such as chat. or functions of chat, such as nicknames, etc

An option I would really like is allowing to only use global chat with a prefix, like "!message here" would act as a shout.
A bit offtopic, but is there some way to share the nicknames with the other servers? eg with essentials

As much configuration as possible, in simpler terms. At the moment I only find use for the tablist, which i've been using a fork of your older plugin GlobalTab to accomplish.

Describe the bug
A clear description of the bug:

When I ping my server in the multiplayer page of minecraft, I am sent a very large Error that references my MOTD.

To Reproduce
Steps to reproduce the behavior:

Simply set up the database info as needed, Let everything be default or even having everything to false or Disabled.

Expected behavior
A clear and concise description of what you expected to happen.

I imagine the spamming of Error was not part of the plan, though this is still in dev so I'm not too worried about it.

Screenshots
If applicable, add screenshots to help explain your problem.

Even better, Here is the link to the Error, of course, shorten down to just the error and not the duplicates.
https://hastebin.com/naruzidage.php

Desktop (please complete the following information):

• Velocity Version: velocity-proxy-1.1.0

• Catalyst Version: Catalyst-Velocity-0.2.1

• Additional Plugins installed on velocity:

Anvil-Velocity-0.2.1
LuckPerms-Velocity-5.1.93

Additional context
Add any other context about the problem here.

In the new version of catalyst where there is a option to pass chat through to the backend server, allow an option to have chat channels that are pass through to still be compared to the disallowed words list and block messages with a disallowed word from being sent to backend chat or make it so that *** for the word is passed instead of the word.

Describe the bug
A clear description of the bug: Players appear above information in tab list

To Reproduce
Steps to reproduce the behavior: Have some players join the game

Expected behavior
A clear and concise description of what you expected to happen.
All Players to be below the information

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

• Velocity Version: Velocity 1.1.8
• Catalyst Version: 0.3.2
• Additional Plugins installed on velocity: anvil, luckperms, skinsrestorer, tcpshield, vbans

Additional context
Add any other context about the problem here.

A clear description of the feature you'd like to see implemented

As stated in the title, I would like to add a backend GUI plugin (like you see on many different servers) that can be accessed by either a hub inventory or via the /server command. Either way, this will require a plugin on the backend servers and utilize either plugin messages or Redis in order to transmit data between the proxy and the backend server. (This will also be toggleable via the config, so you don't have to have this plugin enabled to transfer between servers)

A clear description of the feature you'd like to see implemented
it would be nice if its possible to add support for seeing player list on discord from a specific Server e.g. !list Server

A clear description of the feature you'd like to see implemented

An option to toggle join/leave messages

A clear description of the feature you'd like to see implemented

Following a discussion from Discord:
Enable support of the proxy commands but server-side, so it can be used via command blocks, or plugins/mods which uses signs, buttons or entities/NPCs to execute commands.
Mainly useful to setup fancier ways to teleport players to other servers for example (instead of just typing "/server servername" or "/send player servername") but I think this implementation would be great in the future in case Catalyst grows with more commands.
Maybe use existing Anvil plugin for Spigot (or other server type) to enable this feature, or make a new bridge plugin.

A command such as /report, that allows players to report a player or bug or anything like that.
Have the report info post the player, coords of where they can the command, player they're reporting.

Maybe two separate commands would be more ideal? /reportplayer and /reportbug?
But yeah here's the idea.

A clear description of the feature you'd like to see implemented

Having a config option to turn off the MOTD offered by Catalyst to use other options such as ServerListPlus.

Hey,

Could you please allow an option to disable this completely? Or allow "" to simply not send anything?

I had both connection & disconnect set to "" and it does just that, sends an empty line when somebody joins/leaves.

• Bug/Feature.

At the moment, you can only see when a player joined the proxy. If they're unable to join a server then you still see the player joined message but they aren't in the server. This can be kind of annoying in some situations.

A way to resolve this could be to verify the player was able to join a server (the default server, lobby, etc). An extension to this would be to show (or configurably show) which server the player connected to in the join message as well as a server switching message eg {player} switched to {server}.

Another unrelated note, theres plenty of issues that have been solved but are still marked as open.

Thanks for your work on this valuable plugin

Describe the bug
After you edit a message sent on discord and edit it, it does not edit nor send the edited version.

To Reproduce
Send a message on discord. Check in-game chat. Edit it. Check in-game chat.

Expected behavior
Edit the sent message in-game as well.
Send a new message with edits.

The plugin DiscordSRV provides functionality to link a discord server with a minecraft server, sync roles with minecraft ranks, etc. I hope this plugin can help replace that on a network-wide level.

Some features I'd like to see:

• Global channel (across multiple discord channels and minecraft servers) -- eg: #general on server A and B would both link to all minecraft servers in the network
• Channels per (minecraft) server -- eg: #general-survival on server A and #general-survival on server B would both link to the survival server
• rank/role sync minecraft<->discord
• idk i had some more but i forgot

The issue
Currently, if you have advancedServerInfo enabled, this disables listing servers in-game using /server. Attempting to connect to a well-known server i.e. /server lobby yields an error.

The feature request
I'd like there to be a toggle in the advancedServerInfo section of the config labelled something like "serverSwitchingDisabled" or even simpler "modpackMode". With this option set to true we would have the existing functionality where servers do not show when /server is executed in-game and attempting to connect to a well-known server i.e. /server lobby will fail as it currently does.
Having this option set to false would still have the server MOTD messages show on ping in the server list screen in the main menu of the game, however, users would now be able to see the other servers when running /server in-game and be able to attempt connecting to any of the servers listed there. This should not alter the effect of checking the server version when ViaVersion support is disabled.

Motivation example
As an admin/owner, I have 2 Forge servers and 2 vanilla servers, I want to display a different MOTD for each server.
As a player on Forge server 1, I want to switch to Forge server 2 without having to return to the server browser since both servers are running the same modpack.
As a player on Vanilla server 1, I want to go to Vanilla server 2 without having to return to the server browser.

A clear description of the feature you'd like to see implemented

When a player pings the proxy from the multiplayer menu, i would like to then check if the player pinging is banned and display a custom message back to them. This message would be fully customizable via the config allowing for the server owner to display a plethora of items, depending on their liking.

Some ideas that can be displayed:

How long until they are unbanned (if temp banned)
Why they are banned

A clear description of the feature you'd like to see implemented

Intergration with SkinRestorer for getting the skin used by the player and using it for the chat webhook

take role colour from discord as the colour for discord user's name in minecraft
[Discord] &e%discordname%
^ mapped from discord role
like that

As per the custom chat censor list, there should be a permission available to bypass this filter in the configuration.

A clear description of the feature you'd like to see implemented

I know the teleport function started to be WIP by the devs, so there's might be some stuff already planned but here's my suggestion:
Being able with a command to not only send a player into a server, but actually teleport directly to a player, and specified locations (warp system) other than the spawn. Optionally, be able to support warp to non-default world of the server (ex: Multiverse-Core support). Warps would be encoded as world coordinates, and could be stored as Yml or SQLite (maybe with a bridge plugin for Spigot/Paper side)

I think this function would be really appreciated for a plugin which wants to give "Essentials" commands.
Allow teleport or warp could also complement the other issue "GUI server selector" functions ?

A clear description of the feature you'd like to see implemented

In order to block spamming of messages that occur when a server is not yet fully loaded and a player joins, there needs to be some sort of system the checks if the server has loaded or not.

Not really a high priority feature, but it'd be nice to have some of the newer emojis. I'm not sure, but I think you might be using the same font/textures that the older plugin EmojiChat by RadBuilder is using.

A clear description of the feature you'd like to see implemented
Have an option to customise or completely remove the nickname indicator would be handy.

Describe the bug
Currently, when someone with the permission for colored nicknames tries to make a nick with "&4Test" the command returns noNickColorPermissions

Desktop (please complete the following information):

• Velocity Version: 1.1.0
• Catalyst Version: RC32
• Anvil Version: RC60

Suddenly, users are no longer able to login:

[11:39:23 INFO]: [connected player] Tester has disconnected: An internal error occurred in your connection.
[11:39:23 ERROR]: [connected player] Tester: exception encountered in com.velocitypowered.proxy.connection.client.ClientPlaySessionHandler@1391bfb
io.netty.handler.codec.EncoderException: java.lang.IllegalArgumentException: Don't know how to serialize org.anvilpowered.catalyst.common.util.CommonArgumentType
	at io.netty.handler.codec.MessageToByteEncoder.write(MessageToByteEncoder.java:125) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.channel.AbstractChannelHandlerContext.invokeWrite0(AbstractChannelHandlerContext.java:717) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.channel.AbstractChannelHandlerContext.invokeWriteAndFlush(AbstractChannelHandlerContext.java:764) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.channel.AbstractChannelHandlerContext.write(AbstractChannelHandlerContext.java:790) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.channel.AbstractChannelHandlerContext.writeAndFlush(AbstractChannelHandlerContext.java:758) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.channel.DefaultChannelPipeline.writeAndFlush(DefaultChannelPipeline.java:1020) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.channel.AbstractChannel.writeAndFlush(AbstractChannel.java:299) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at com.velocitypowered.proxy.connection.MinecraftConnection.write(MinecraftConnection.java:197) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at com.velocitypowered.proxy.connection.backend.BackendPlaySessionHandler.lambda$handle$2(BackendPlaySessionHandler.java:193) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at java.util.concurrent.CompletableFuture$UniAccept.tryFire(CompletableFuture.java:714) ~[?:?]
	at java.util.concurrent.CompletableFuture$Completion.run(CompletableFuture.java:478) ~[?:?]
	at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:164) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:472) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:384) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:989) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at java.lang.Thread.run(Thread.java:834) [?:?]
Caused by: java.lang.IllegalArgumentException: Don't know how to serialize org.anvilpowered.catalyst.common.util.CommonArgumentType
	at com.velocitypowered.proxy.protocol.packet.brigadier.ArgumentPropertyRegistry.serialize(ArgumentPropertyRegistry.java:89) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at com.velocitypowered.proxy.protocol.packet.AvailableCommands.serializeNode(AvailableCommands.java:148) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at com.velocitypowered.proxy.protocol.packet.AvailableCommands.encode(AvailableCommands.java:111) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at com.velocitypowered.proxy.protocol.netty.MinecraftEncoder.encode(MinecraftEncoder.java:34) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at com.velocitypowered.proxy.protocol.netty.MinecraftEncoder.encode(MinecraftEncoder.java:12) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	at io.netty.handler.codec.MessageToByteEncoder.write(MessageToByteEncoder.java:107) ~[1.1.x-SNAPSHOT.jar:1.1.3-SNAPSHOT (git-aef0e4a8-b329)]
	... 17 more

I would love to have a console channel option. I don't use staff chat (That's what discord is for) but being able to see and run commands without needing to login to the box is very helpful.

This is a feature of DiscordSRV.

Two players can have the same nickname

Describe the bug
Send msg in console take error :

To Reproduce
console : staffchat test

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots

[catalyst]: Console executed command : staffchat truc
[20:12:13 ERROR] [com.google.common.eventbus.EventBus.default]: Exception thrown by subscriber method onStaffChatEvent(org.anvilpowered.catalyst.api.event.StaffChatEvent) on subscriber org.anvilpowered.catalyst.common.listener.CommonStaffChatListener@6bebd79d when dispatching event: org.anvilpowered.catalyst.common.event.CommonStaffChatEvent@22ee4bb5
java.lang.NullPointerException: Cannot invoke "com.velocitypowered.api.proxy.Player.getUsername()" because "player" is null

Desktop (please complete the following information):

• Velocity Version: latest
• Catalyst Version: latest

A clear description of the feature you'd like to see implemented
It would be nice to see if its possible to send a Message into the Chat automatically when a Server goes offline/online.

Describe the bug
A clear description of the bug:

if you use a shop plugin you cannot use the commands because nothing happens no error no change

https://ibb.co/f9XGpHv

the same problem also exists if you use a free plugin like this: QuickShop Reremake - 1.16 Ready - Say hello with RGB! 4.0.5.9

A clear description of the feature you'd like to see implemented

Having the ability to set up Command Aliases like this plugin;
MyCommand Bungee

Being able to similarly work it so you can execute commands such as "/server testServer1" like /testserver for example.