Andrew Steinborn
5303239f66
fix bug
2021-08-21 02:23:29 -04:00
Andrew Steinborn
e6a93ad0c6
Clean up JoinGame packet encoding/decoding
...
Specifically, the JoinGame packet handling is now split between pre-1.16 encodings of the packet and post-1.16 handlings of the packet. This packet is one of the most amorphous packets in the entire Minecraft protocol, from Velocity's perspective.
2021-08-21 02:17:34 -04:00
Andrew Steinborn
65db0fad6a
Fix bug
2021-08-17 13:39:35 -04:00
Andrew Steinborn
6d4cb3b18f
Fix TabList population
...
Bug reported by JakubBien on Discord
2021-08-17 13:30:15 -04:00
Corey Shupe
ec4d2d45a1
Make Mojang Sessionserver Configurable ( #560 )
2021-08-17 13:14:39 -04:00
Andrew Steinborn
d3ff6f8e33
Make sure to issue a backpressure-solving flush on a future iteration of the event loop
...
Thanks to @Spottedleaf for finding this issue.
2021-08-04 23:37:27 -04:00
Andrew Steinborn
c32bc88064
Fix not forwarding channelReadComplete events in some cases
2021-08-04 23:01:28 -04:00
Andrew Steinborn
d3e1b358fb
Fix repeating "you were kicked from X" message
2021-07-30 15:42:34 -04:00
Andrew Steinborn
211da180b3
Revert 2124b5ec, suspect that event firing may be broken
2021-07-28 13:05:50 -04:00
David Mayr
3d8e9091c0
Custom localization for each player ( #537 )
2021-07-25 21:50:44 -04:00
Andrew Steinborn
2124b5ec12
Cache all types in EventTypeTracker
2021-07-24 18:30:13 -04:00
Andrew Steinborn
62d234bb26
Improve event system tests
...
Sleeping in a test is considered bad form. Replace the delay with checks to ensure that threads are spawned appropriately and a new test to ensure that continuation threads do run before other listeners.
2021-07-24 18:15:19 -04:00
Hugo Manrique
76c6827926
Expose registered aliases in API ( #549 )
2021-07-24 14:34:50 -04:00
Andrew Steinborn
2f82603f65
We don't need the sorted order that a List provides
2021-07-19 13:33:03 -04:00
Andrew Steinborn
349781cff8
Fix licensing stuff
2021-07-19 13:29:33 -04:00
Andrew Steinborn
e017949abf
Fix #547 and and #548
...
The first bug in #548 (and the only issue in #547 ) was a typo. The second bug was fixed by tracking "friends" of the event types, so we can invalidate everything as needed.
2021-07-19 13:26:37 -04:00
Andrew Steinborn
f79736b8b7
Fix event ordering.
2021-07-12 08:37:26 -04:00
Andrew Steinborn
31aa996623
Minor server-switching code cleanup
2021-07-10 23:36:25 -04:00
Andrew Steinborn
ea577019b8
Don't force-add a tab list entry if it already exists
2021-07-09 10:11:15 -04:00
Andrew Steinborn
0e0a14498f
Fix some localization and API issues
...
Resolves #536
2021-07-08 03:08:02 -04:00
Andrew Steinborn
540d970a4d
Minecraft 1.17.1 support ( #530 )
2021-07-06 03:32:13 -04:00
Andrew Steinborn
1f7d14f6d1
Checkstyle and friends
2021-06-27 21:18:52 -04:00
Andrew Steinborn
254e8f1a1e
Use the "closest" locale from Velocity
2021-06-27 21:15:17 -04:00
Andrew Steinborn
c6ef84eb7f
Backport localization support to Velocity 3.0.0 from Polymer
2021-06-27 20:04:43 -04:00
Andrew Steinborn
3b6b73f216
Make announce-player-commands not suggest aliases
...
Fixes #533
2021-06-24 10:10:34 -04:00
Andrew Steinborn
59ca9a534d
Ensure that the in-flight connection is always reset.
2021-06-20 17:36:44 -04:00
Andrew Steinborn
d90952ca3a
Fix licensing again
2021-06-14 14:00:35 -04:00
Andrew Steinborn
c979c61155
Add some internal injection points for the benefit of ViaVersion
...
See #524 for more details
2021-06-14 13:59:56 -04:00
Andrew Steinborn
93724b7b33
Do not emit errors for kicks from servers
2021-06-14 00:04:18 -04:00
Andrew Steinborn
1dfe294b96
Remove GenericArgumentPropertySerializer
...
This was only used for boolean argument tyoes
2021-06-13 06:55:24 -04:00
Andrew Steinborn
5a337afb00
Include compatible protocol versions in the incompatible version message
...
Fixes #519
2021-06-13 05:17:09 -04:00
Andrew Steinborn
83dff50bc7
Merge pull request #520 from hugmanrique/feat/3-command-impl-cleanup
...
[3.0] Command system implementation refactor
2021-06-13 09:12:06 +00:00
Andrew Steinborn
ebd0fcc6b1
Remove unnecessary slicing of uncompressed packets
2021-06-13 04:00:48 -04:00
Andrew Steinborn
c33f9e0466
Merge remote-tracking branch 'origin/dev/3.0.0' into dev/3.0.0
...
# Conflicts:
# api/src/main/java/com/velocitypowered/api/proxy/Player.java
# proxy/src/main/java/com/velocitypowered/proxy/connection/client/ConnectedPlayer.java
2021-06-12 20:13:52 -04:00
Andrew Steinborn
c2b82f878b
Merge branch 'dev/1.1.0' into dev/3.0.0
...
# Conflicts:
# api/src/main/java/com/velocitypowered/api/proxy/Player.java
# proxy/src/main/java/com/velocitypowered/proxy/connection/client/ConnectedPlayer.java
2021-06-12 20:12:31 -04:00
Andrew Steinborn
1b32f56168
Merge pull request #523 from Artuto/dev/1.1.0
...
Add support for retrieving the Player's client brand
2021-06-12 23:11:41 +00:00
Artuto
f2b20e6398
Remove usage of Optional
2021-06-12 13:48:42 -05:00
lexikiq
72408d5bd8
Remove display name
2021-06-12 12:12:04 -04:00
Andrew Steinborn
b0da4b60c7
Do not check if a plugin message channel is registered before forwarding it
...
This fixes issues with Servux and probably a crap ton of other bad mods. Purity on this front serves no real purpose than to say we're pedantic.
A future commit may remove all tracking of registered channels altogether. There's strong evidence (back to Minecraft 1.8, at least) that we don't have to track this data and only need to send channels for plugin message channels registered by Velocity itself. This will likely involve more testing to see if this uncovers client bugs with older versions of the game.
2021-06-12 10:30:46 -04:00
Artuto
c8d50f3fdd
Add support for retrieving the Player's client brand
2021-06-11 23:44:51 -05:00
lexikiq
8356b812c3
Cache permission pointers
2021-06-11 23:44:59 -04:00
lexikiq
4003666c27
Cache pointers
2021-06-11 23:32:49 -04:00
lexikiq
19f80bd618
Don't directly implement PermissionChecker
2021-06-11 21:49:40 -04:00
lexikiq
29b409b9f1
Move #pointers to Player
...
It uses fully API methods so it makes sense to me to use it in Player, open to criticism though
2021-06-11 21:33:15 -04:00
lexikiq
db7e3262a6
Optimize imports
2021-06-11 21:25:51 -04:00
lexikiq
3c4312056b
Switch to #pointers
...
Ensures that Pointered#getOrDefault[From] works
2021-06-11 21:12:44 -04:00
lexikiq
f59516165f
Rebase onto feature/adventure-pointers
2021-06-11 21:06:59 -04:00
Hugo Manrique
61480544f5
Apply suggested future-related changes
...
Co-authored-by: A248 <theanandbeh@gmail.com>
2021-06-11 14:03:38 +02:00
Hugo Manrique
f6e6f02a84
Fix checkstyle violations
2021-06-10 20:35:36 +02:00
Hugo Manrique
fb7aafe8ae
Fix Brigadier command alias redirects
2021-06-10 20:25:52 +02:00
Hugo Manrique
b18e78eac7
Add more tests and clean up
2021-06-10 20:00:33 +02:00
Riley Park
f579725002
Provide access to Pointer values
2021-06-10 01:08:27 -07:00
Hugo Manrique
ba34e4729b
Add more suggestion tests
2021-06-09 22:22:15 +02:00
Hugo Manrique
bcb68c8d0f
Test suggestion exception handling
2021-06-09 21:00:17 +02:00
Hugo Manrique
a9d40f3ca3
Fix alias typo in suggestion merge test
2021-06-09 19:15:27 +02:00
Andrew Steinborn
445688b9a1
Merge branch 'dev/1.1.0' into dev/3.0.0
...
# Conflicts:
# build.gradle
2021-06-09 01:29:51 -04:00
Andrew Steinborn
e1af30e0fa
whoops
2021-06-08 15:42:20 -04:00
Andrew Steinborn
8892e2a5c6
We don't need the hash to go along with the resource pack response
2021-06-08 15:41:54 -04:00
Andrew Steinborn
a65f720f9d
Merge branch 'dev/1.1.0' into dev/3.0.0
...
# Conflicts:
# build.gradle
2021-06-07 15:32:52 -04:00
Andrew Steinborn
496372d7af
Check if the resource pack hash looks like it might be a SHA-1 hash
2021-06-07 15:32:36 -04:00
Hugo Manrique
46b1bee83a
Test suggestions
2021-06-07 15:15:48 +02:00
Hugo Manrique
ea716c15c6
Test CommandManager
2021-06-07 14:36:41 +02:00
Hugo Manrique
d429d8383d
Test execution and injection
2021-06-07 14:13:32 +02:00
A248
6e42f225e2
Bump to Caffeine 3
2021-06-06 08:09:42 -04:00
Andrew Steinborn
ef7aeae4ed
Merge remote-tracking branch 'origin/dev/3.0.0' into dev/3.0.0
2021-06-06 03:52:03 -04:00
Andrew Steinborn
9d54a03a4b
Force SpotBugs to shut up here
...
This method is indeed called...
2021-06-06 03:51:42 -04:00
Andrew Steinborn
b1b3882fab
Merge branch 'Xernium-future/1.17' into dev/3.0.0
...
# Conflicts:
# api/src/main/java/com/velocitypowered/api/proxy/ProxyServer.java
# proxy/src/main/java/com/velocitypowered/proxy/connection/client/ConnectedPlayer.java
2021-06-06 03:48:49 -04:00
Hugo Manrique
ff506dfdf8
Add license headers
2021-06-05 23:20:01 +02:00
Hugo Manrique
703b91e0fa
Command implementation refactor
2021-06-05 23:19:01 +02:00
Hugo Manrique
ff504c21ef
Suggestions provider
2021-06-05 18:31:00 +02:00
FivePB
59b29a075e
Improve new Resource-pack API
2021-06-05 16:07:24 +02:00
Andrew Steinborn
7fbfdb2f23
Merge pull request #513 from CoreyShupe/feature/Allow-Raw-Registered-Server-Creation
...
Initial implementation of raw registered server creation.
2021-06-01 13:12:41 +00:00
CoreyShupe
61c0c0d083
Initial change to allow for server info equality.
2021-06-01 02:17:35 -04:00
CoreyShupe
150808b4b0
Initial implementation of raw registered server creation.
2021-06-01 02:07:06 -04:00
A248
2cfa3f743f
Remove Add-Opens from manifest
2021-05-24 12:02:35 -04:00
Andrew Steinborn
64d30fba81
AsyncEventExecutor -> AwaitingEventExecutor
2021-05-23 17:51:07 -04:00
Andrew Steinborn
e14ea36150
We don't rely on kyori/event any more
2021-05-23 15:52:09 -04:00
Seppe Volkaerts
821ca02ee7
Backport Velocity Polymer's async event API, with changes.
...
This commit backports the event manager from Velocity Polymer, with some changes for Velocity 1.1.x API compatibility:
- All event handlers run asynchronously. (While EventTask.async() exists, it is not useful in 3.0.0, but is provided as a migration aid for Polymer.)
- Event ordering is currently limited to the 5 levels available in Velocity 1.x.x.
2021-05-23 15:39:06 -04:00
Andrew Steinborn
3f50964f36
Fix CommandManager method naming so that it is in sync with the 1.1.0 API
2021-05-23 14:20:59 -04:00
Andrew Steinborn
977300cc66
Remove text 3.x API and associated deprecated APIs
2021-05-23 14:13:51 -04:00
Andrew Steinborn
569bb4a16b
Just import the entire command manager from Polymer, without changing any names
2021-05-23 13:58:27 -04:00
Andrew Steinborn
590f18dbe0
Convert to Java 11 compressor
2021-05-23 13:58:01 -04:00
Andrew Steinborn
d6dcb115f1
Remove remaining deprecated APIs
2021-05-23 13:49:06 -04:00
Andrew Steinborn
6816c15385
Remove bad test
2021-05-23 13:47:20 -04:00
Andrew Steinborn
2f0ee15051
Remove deprecated Velocity 1.0.0 Command API.
2021-05-23 13:46:47 -04:00
FivePB
18466bb595
Snapshot 21w19a
2021-05-13 23:50:00 +02:00
FivePB
8def411b2b
Snapshot 21w15a
2021-05-13 23:49:55 +02:00
Five (Xer)
581303a54c
Snaphot 21w10a
2021-05-12 21:42:27 +02:00
Five (Xer)
6207647aae
Remove leftovers
2021-05-12 21:42:27 +02:00
Five (Xer)
22bc759044
Snapshot 21w08a
2021-05-12 21:42:27 +02:00
Five (Xer)
b19cf16345
Snapshot 21w07a
2021-05-12 21:42:27 +02:00
FivePB
f0d9a445ed
Snapshot 21w03a
2021-05-12 21:42:27 +02:00
Five (Xer)
040cc29c34
Snapshot 20w49a
2021-05-12 21:42:27 +02:00
Five (Xer)
d8a39fc438
Snapshot 20w46a
2021-05-12 21:42:27 +02:00
Five (Xer)
7f0964155c
Snapshot 20w45a
2021-05-12 21:42:27 +02:00
Andrew Steinborn
11ed4b46e4
whoops
2021-05-09 02:57:52 -04:00
Andrew Steinborn
5170da8fd7
Improve writeVarInt inlining by peeling the two most common cases
2021-05-09 02:57:01 -04:00
Andrew Steinborn
150fd9a9cf
Add highly-optimized VarInt writing method
2021-05-08 23:26:43 -04:00
Andrew Steinborn
6369a95ec9
Readd safe and slow compression handling and hide it behind a system property
2021-05-08 18:40:23 -04:00
Leymooo
a8e0516d18
Also do not copy memory in case when packet needs to compress
2021-05-07 19:36:30 +03:00
Andrew Steinborn
37a4199d43
Combine VarInt prefix encoding with compression
...
This saves us a memory copy in the common "there is no need to compress this packet" case.
2021-05-06 19:56:45 -04:00
Andrew Steinborn
fb3f21abc6
More bitshifting magic
2021-05-06 00:48:19 -04:00
Andrew Steinborn
1cef82d54d
Unroll the VarInt writing loop
...
This is about as optimized as it can get. Thanks to @Leymooo for the idea, I simply expanded on it. We optimize for the common 1-3 byte cases, and punt more "complicated" cases to the original VarInt writing function we had before.
2021-05-05 22:13:54 -04:00
Andrew Steinborn
d42cc4f984
Force a flush after a certain threshold of packets have been queued for sending
...
Fixes #486 .
2021-05-04 18:51:01 -04:00
Nicolas RAYNAUD
e0153267db
Fixed copyright year ( #490 )
2021-05-04 18:22:50 -04:00
Andrew Steinborn
7c76ae9a27
Make sure to check not only the entire array but also how much was written
2021-05-04 16:32:37 -04:00
Andrew Steinborn
3dc8e25ec7
Fix varint writing (for real!)
...
Thanks to @Leymooo for reporting and providing test cases. (I also added one of my own tests which proved the most useful to debugging the issue.)
2021-05-04 16:30:32 -04:00
Riley Park
2713831f77
Revert "Avoid calling writeVarInt in the (very) common uncompressed packet case"
...
This reverts commit c041bea1b6
2021-05-03 19:38:39 -07:00
Andrew Steinborn
c041bea1b6
Avoid calling writeVarInt in the (very) common uncompressed packet case
2021-05-03 20:17:51 -04:00
Andrew Steinborn
0debb81392
I was nagged to make it more consistent
2021-05-03 20:08:16 -04:00
Andrew Steinborn
0811ebb312
Gotta add the license header to the test file
2021-05-03 19:18:06 -04:00
Andrew Steinborn
4ca97a6df9
Reapply "Optimize varint writing"
...
Inspired by the approach described at the bottom of https://richardstartin.github.io/posts/dont-use-protobuf-for-telemetry
Given that we do a lot of varint writing as well, this should provide a small performance boost for larger/complex packets whilst not regressing hard on smaller packets.
This includes a test to ensure that the behavior is as expected and fixes the initialization loop so that the correct results will be given. Much thanks to @octylFractal for acting as my duck while trying to figure this out.
2021-05-03 19:14:48 -04:00
Andrew Steinborn
e531cdb373
Revert "Optimize varint writing"
...
This reverts commit 3db2fe8d63
2021-05-03 18:07:25 -04:00
Andrew Steinborn
3db2fe8d63
Optimize varint writing
...
Inspired by the approach described at the bottom of https://richardstartin.github.io/posts/dont-use-protobuf-for-telemetry
Given that we do a lot of varint writing as well, this should provide a small performance boost for larger/complex packets whilst not regressing hard on smaller packets.
2021-05-03 17:31:32 -04:00
Andrew Steinborn
347853b945
Disable hinting test for now
2021-04-25 10:18:58 -04:00
Andrew Steinborn
81311e7516
Don't attempt to decode empty buffers ( fixes #482 )
...
Stuff like this makes me want to drop support for <=1.13 versions of Minecraft.
2021-04-25 10:14:13 -04:00
Andrew Steinborn
ce26ae76c2
Fix case-sensitivity for ping passthrough
2021-04-22 11:23:42 -04:00
Andrew Steinborn
ca52f8989e
Load forced hosts as lowercase too, see #464
2021-04-18 08:22:36 -04:00
Andrew Steinborn
ff54622663
Forced hosts should be case-insensitive. Fixes #464
2021-04-18 08:20:04 -04:00
Andrew Steinborn
f1cb3eb1a2
Move timeout handler to after frame decoder
...
Mitigates attacks like the one described in SpigotMC/BungeeCord#3066 . This cannot be considered a full protection, only a mitigation that expects full packets. The attack described is essentially the infamous Slowloris attack.
2021-04-15 22:56:37 -04:00
Andrew Steinborn
c51359e893
Move away from deprecated percent() method.
...
Also fixed another small nit.
2021-04-15 04:23:59 -04:00
Andrew Steinborn
4de71fe942
Fix color and overlay packets for Adventure boss bar
...
Someone discovered this bug but decided not to tell anyone. For the record, I'm the one who was brain-damaged... well, I made that particular copy-paste error.
2021-04-15 04:22:44 -04:00
Cubxity
5ea6728d1a
Added ListenerBoundEvent and ListenerCloseEvent. ( #454 )
2021-04-02 13:24:45 -04:00
Rocco
c34aee76ae
Fix Forge/Client Proxy Connections mismatch in config ( #456 )
2021-04-02 12:36:32 -04:00
A248
7ba2318506
Fix one more case of a possibly swallowed exception ( #455 )
...
Maybe spotbugs will add a rule for this in the future
2021-04-02 01:45:58 -04:00
Andrew Steinborn
c747120b20
Relicense the Velocity proxy under the terms of GPLv3 ( #448 )
2021-03-31 16:04:06 -04:00
Andrew Steinborn
514115a85c
Do not use an ASCII string for handshake
2021-03-30 12:10:55 -04:00
Andrew Steinborn
f88283f127
Revert "Revert strict handshake hostname checks"
...
This reverts commit 4f80d2b261
2021-03-30 12:08:08 -04:00
Andrew Steinborn
8aad6e2ece
clear input buffer in case of bad varints
2021-03-30 12:05:43 -04:00
Andrew Steinborn
0cc300b621
don't swap these two around
2021-03-23 07:38:07 -04:00
Andrew Steinborn
3111816545
Warn about null permission functions.
2021-03-23 07:32:01 -04:00
A248
0016f4ce19
Relay more exceptions ( #446 )
2021-03-22 08:11:25 -04:00
Andrew Steinborn
015693090f
Fix typo (thanks zml)
2021-03-21 16:17:41 -04:00
Andrew Steinborn
da54a091fb
Allow the java.lang module to be opened, as a temporary Java 15+ workaround
2021-03-21 16:16:19 -04:00
Andrew Steinborn
72d47b5a3d
Don't advertise the BungeeCord plugin messaging channel if it is disabled.
2021-02-18 18:12:51 -05:00
David Mayr
648624d333
Check permissions before providing suggestions ( #430 )
2021-02-16 19:13:48 -05:00
Andrew Steinborn
1a768bda9d
We're going to need custom hash maps after all.
2021-02-16 01:15:58 -05:00
Andrew Steinborn
b94303d2be
Map command nodes being serialized by their identities
...
Fixes #429
2021-02-16 01:14:22 -05:00
Bastian
7a713e9379
Update bStats and migrate to new config file
...
This commit simplifies the Metrics class by using the new base module
from bStats. It also migrates to the new bStats config file that will
be used by plugins that integrate bStats. If a user disabled bStats in
the old config file, this setting will be copied to the new config file.
2021-02-12 20:29:07 +01:00
Camotoy
fb879cb498
Apply initial requested changes
2021-02-09 14:13:33 -05:00
Camotoy
d47b339908
Add PlayerPluginMessageRegisterEvent
2021-02-09 13:51:43 -05:00
Andrew Steinborn
7e42c5b2e7
Turn some APIs into no-ops if they apply to clients without the relevant client functionality.
2021-02-07 15:19:59 -05:00
Andrew Steinborn
b88c573eb1
Correctly forward the player's virtual host to the remote server.
2021-02-01 16:17:02 -05:00
Andrew Steinborn
4f80d2b261
Revert strict handshake hostname checks
...
These will break TCPShield and Cosmic Guard plugins. Sad as this was a worthwhile mitigation.
2021-01-29 23:08:14 -05:00
Andrew Steinborn
501853e807
Fix Checkstyle issue for the umpteenth time...
2021-01-29 17:59:00 -05:00
Andrew Steinborn
4f6d238b39
Strictly limit the hostname size and limit it to ASCII characters only
2021-01-29 17:56:50 -05:00
Andrew Steinborn
4219bf7b09
*Actually* fix Checkstyle issues
2021-01-26 20:02:07 -05:00
Andrew Steinborn
b982c5b6ed
Fix Checkstyle issues
2021-01-26 19:59:29 -05:00
Andrew Steinborn
0c90e94e85
fix
2021-01-26 19:50:43 -05:00
Andrew Steinborn
5ceac16a82
Add upfront size checks for some packets.
...
This is simply a further protection against certain attacks which send malformed packets to the proxy.
2021-01-26 12:33:35 -05:00
