Add stub for Find Lobby endpoint

Co-Authored-By: Niko233 <139348239+NikoCat233@users.noreply.github.com>

Stop hardcoding the GameOptions in GameListingV2

To prevent needing to pull in Impostor.Hazel, use an extension method
instead.

Thanks Niko for researching this part

Co-Authored-By: Niko233 <139348239+NikoCat233@users.noreply.github.com>

Bump minor version

At this point it doesn't seem necessary to drop support for old
Amoguses, so just bump the minor version for support purposes

Forbid crossplay between 2024.10.29 and 16.0

Innersloth also forbids it, and it's likely not possible due to the new
GameOption version

Add endpoint to find games by code

This is used in the new "Enter Code" flow

3 extra fields are added to GameListing, including a Base 64
serialization method for game options

Update Game Options

Creating a game is now possible

Add new network version

Add diagnostic http message (#663)

* Add diagnostic message to test HTTP server

This makes it possible to diagnose if the reverse proxy setup is working
correctly or if the game configuration is wrong

* Print console log as well

Only print it once to prevent console spam

* Update src/Impostor.Server/Http/HelloController.cs

Co-authored-by: js6pak <me@6pak.dev>
---------

Co-authored-by: js6pak <me@6pak.dev>

Add missing cosmetic methods to IInnerPlayerControl

Push down AC checks in PlayerPhysics as well

I've checked Impostor for issues that are similar to #664, and found
this one: if mods add a custom rpc on PlayerPhysics, Impostor will check
ownership of it, which isn't intentional.

Move down target AC check in InnerShipStatus

Mods may send RPC's on a ShipStatus that aren't broadcasts. Previously
Impostor would forbid that, which doesn't make sense to me. By moving
down the check it'll allow mods to broadcast RPC's on ShipStatus.

Reported-By: xtracube

Bump version to 1.10.2

Check for disconnection between handling messages

Without this, it is possible for clients to get the remainder of their
messages handled in a packet.

Co-Authored-By: js6pak <me@6pak.dev>

Clean up abandoned games on disconnect

When a client disconnects without ever having joined a game, it leaves
behind an empty game on the server. Remove these when a player
disconnects.

Co-Authored-By: js6pak <me@6pak.dev>

Prevent clients from creating a second game

Legitimate clients will never create a second game using the same
connection.

Co-Authored-By: js6pak <me@6pak.dev>

Bump to Among Us 2024.10.29 (#656)

Inspected mono, no networking changes found.

Ran dumpostor as well

Bumped Impostor version

Bugfix/fix outfit RPCs (#653)

* implemented the sequence ID

* implemented the sequence ID for the player outfits

the sequence ID in the set call must be greater than that stored on the client otherwise the client disregards the set call.

* make sequence IDs internal

* added space after closing brace

* make the sequence ID addition +5

Remove unused HandleSetInfected

updated the api call to use the setnamerpc serialise

the client now expects a byte which is the netid otherwise the setname call wont work

Fix vanilla venting as Phantom cause a ban

Add support for 2024.9.4

Fix RemovePlayer reason on internal disconnects

Send DisconnectReason.Hacking to other players when disconnecting for cheating

Base game doesn't have a specific message for it so it results in a generic "left the game due to error" but it's still better than the default "left the game".
This also allows client side mods to fix the message.

Fix cheating disconnect message

DisconnectReason.Hacking has been broken in base game for a long time so replace it with a custom message.
Also add a random support code for easier debugging in case of false positives.

Refactor SetColor anticheat

One issue with the ColorLimits checks is that it didn't account for mods
that add extra colors, this is fixed by checking if the route followed
is plausible instead of mimicking client logic and only accepting a
single answer that it expected. A false negative is possible where the
host decides to wrap back to the front but there are still custom colors
available at the end of the list. This is acceptable because Impostor
doesn't currently know the amount of colors available and this issue
isn't major enough to ask all mod authors to declare their custom color
amount to the server.

As extra colors should now be handled by this code, it makes sense to
change the category of the unknown colors check to ProtocolExtensions.
For these mods an exemption of the ColorLimits category is no longer
necessary.

Note that mods that allow duplicate colors that do not use a custom rpc
to set colors can only be supported by disabling the ColorLimits
category, this was already the case and I don't think it can be unified
with the existing check.

更新 Game.State.cs

Merge pull request #609 from Impostor/feature/2024.6.18-prepare

Prepare for 2024.6.18 update

Add handler for phantom rpcs

Add Phantom Rpcs support

Add Phantom to the list of Impostor roles

Clean up owned playerinfo's when disconnecting

This commit does what official servers do:
- despawn the PlayerInfo if someone disconnects while in the lobby
- despawn all PlayerInfos at the end of a game when the EndGame screen
is shown. At this point a lot of other netobjects are destroyed as well,
with the PlayerInfos generally done first.

Set PlayerLevel in PlayerInfo on RPC SetLevel

Previously this used to be done by the host, but this is now the
server's responsibility.

Add Message24EndGameHostMigration serialization

Looks like Innersloth added this but hasn't actually used this. Weird..

When sent it makes the new host perform a full rejoin, which gums up
Impostor's spawning logic. Fixing that causes the client to gum itself
up. Sigh.

After some testing it seems like host migration still works fine, so
this investigation wasn't needed. Committing for posterity.

Co-Authored-By: js6pak <me@6pak.dev>

Switch PlayerName writes to Outfit

In 2024.6.18 Innersloth changed the storage location of the PlayerName
from PlayerInfo to PlayerOutfit. If we don't set the PlayerName, we'll
send an incomplete PlayerInfo to new players, so new players are unable
to stay connected as they will not receive complete PlayerInfo objects
before a timeout in the client activates.

Because we publish PlayerName in the API, we can't completely remove
the PlayerName field from PlayerInfo. We also use it internally in a few
places, so I don't this is worth removing.

Sync over tasks

As tasks used to be part of the GameData, 2024.6.18 has moved them to
the PlayerInfo. Because they are now called on the PlayerInfo of the
correct player, the playerId field in the RPC is no longer necessary.

Bump to 2024.8.13

Network side seems unchanged and officials allow crossplay

Spawn PlayerInfo's if host is not authoritive

This PlayerInfo also needs to be synced over if it is server owned

Add NetId to SetName and SetColor

2024.6.18 adds a NetId field *in front* of the existing fields to help link back to the PlayerInfo. We don't actually need this NetId and assume it's called on the correct PlayerControl, so we ignore it.

Set RoleWhenAlive when set to a Ghost role

Serialize Color -1 as 0 in PlayerOutfit

Official servers send 0 instead of -1 when the outfit is still
incomplete to prevent some client issues.

Demote InnerGameData, promote InnerPlayerInfo

In 2024.6.18 Innersloth promoted PlayerInfo to its own network object, and gave it to the server. InnerGameData is now a container that holds these players. This commit implements the changes necessary to handle these separate PlayerInfo network objects.

Add dirty tracking to PlayerOutfit

To prevent excessive modifications in the codebase, which is prone to
issues if a callsite is missed, automatically do this using properties.

Keep the access modifiers intact for now.

Add sequence id's to playeroutfit

Add canOverrideRole param to SetRole

This is currently unused by Innersloth

Implement GameOptions V8

This adds a field for RulesPresets and SpecialGameModes, as well as
RoleOptions for the new roles.

VoteBanSystem now contains itself as a component

This makes it consistent with the other spawnables.

Update Dumpostor to v1.0.1 and run for 6.18

Add the RulesPresets and SpecialGamemodes enums

Prepare for 2024.6.18 update

Major changes are incoming, this doesn't yet work.

Add scene param to forbidden scene log statement

I'm seeing multiple warnings in my server log about forbidden scene
changes. Unfortunately this warning does not include the actual scene
that was triggering the issue, which makes diagnosis/remediation difficult.

Additionally change this logging call to use names suitable for
structured logging.

Change default ListenIp to match with config

The default ListenIp for the base game server has been set to 0.0.0.0 in
config.json for as long as I remember, and a different value is only
sporadically used. I remember a case where someone was fancy with Split
DNS. 0 is better as a default than localhost, so change the code
accordingly to aligh with config.json

Modify config.json GameRecorderEnabled Default Value

Update docs/Server-configuration.md File Description

Delete full-config-example.json and merge it into config.json

Bump System.Text.Json to 8.0.4

Fixes CVE-2024-30105 (High 7.5/10)
https://github.com/advisories/GHSA-hh2w-p6rv-4g7w

Impostor doesn't use the affected
JsonSerializer.DeserializeAsyncEnumerable method as it doesn't accept
JSON formatted payloads, but the NU1903 warning is shown on all builds,
leading to warning fatigue.

Let hosts handle exiles

Previously Impostor liked to kill exiled players server side to raise an
event. This is quite early, as the host will SetRole them to a ghost
role later.

However as PlayerInfo is server-owned by default in 2024.6.18, it starts
killing players client side. This caused meetings to immediately
conclude if the final impostor was voted out, ruining the cutscene.

So this commit makes Impostor wait for hosts to actually kill the player
using SetRole. Do set the LastDeathReason property though for players
using the API.

Make Start/Finish GameData API public

Refactor Start/FinishRpc to add GameData methods

Rpc's are a specific sort of GameData packets, so it makes sense to
split and extract a method for sending GameData packets from these functions.

Don't DC the client if they send an unknown flag

This item is only used in a LogTrace call, so it seems excessive to kick
a player on servers just for a log statement

Co-Authored-By: js6pak <me@6pak.dev>

Add compatibility option to allow host authority

There are various bugs in the 2024.6.18 implementation of host-authoritive
multiplayer such that I can't recommend players to use it. Fixing these
issues on the server side is impossible, and I can't smell remotely if
clients have applied the necessary fixes. The currently known bugs
include at least:

- Unable to join more than 8 players in a room
- Unable for new players to join if host switched from host authoritive
  to server authoritive
- Host is unable to kill if it is in a host authoritive game while it
  thinks it is in server authoritive mode

If Innersloth fixes this mode in future game patches, this option may be
reconsidered.

Add new option to full config example

Some people were confused by config-full.json, thinking that this file
should be edited too. This is not the case and the values in this file
don't apply as it was only meant as an example, so rename the file to
make that more clear.

Also update this file to add the options from the Modular Anticheat

Merge pull request #599 from miniduikboot/fix/compatman-no-invalid-clients

Improve CompatMgr behaviour for unknown versions

Improve CompatMgr behaviour for unknown versions

Currently, Impostor will show an "Client is in an invalid state."
message if a player tries to join a match where the host or the joining
player is using a version unknown to Impostor.

This message confused a *lot* of people, so this PR changes the API such
that if the version is not registered, the version itself is put in a
compatibility group and compared, assuming no other game versions are
compatible with it. This works fine in most cases, as Among Us network
versions generally increase over time.

Failure on 2222.2.2 (which is ancient at this point, so likely no longer
used) and future cases where Innersloth messes up (which is unlikely) is
an acceptable consequence to me.

Merge pull request #598 from miniduikboot/fix/custom-rpc-gamelogic

Accept RPC's called on GameManager

Accept RPC's called on GameManager

Previously Impostor would just throw when a method was called on a
GameManager, but this caused issues with client mods that did just this,
even through vanilla never calls an RPC on GameManager.

This commit changes that throw statement to pass the exception to the
custom message handler/anticheat instead, which is also how it works for
the other NetObjects

Merge pull request #597 from Impostor/feature/2024.6.4

Add 2024.6.4 game version

Bump Impostor version

Add 2024.6.4 game version

Officials allow crossplay and after inspecting the Mono I don't see any
major changes, so I'm following officials here

Prepare release 1.9.2

Merge pull request #596 from miniduikboot/fix/cleanup-disposed-clients

Clean up clients that refuse to quit properly

Clean up clients that refuse to quit properly

When a player is kicked or banned from a game, the client is expected to
disconnect in response. When they decide not to *or* the disconnect
packet is lost, Impostor just silently disposes of the connection.
This does not remove them from ClientManager, which is how I tracked
this bug down.

By sending them a proper disconnect message, clients should have
received *a* disconnection message before dropping their connection.

Future work should split this entire method in removing players from a
game and processing kick/bans because the latter feature is not relevant
for other callers of this method.

Merge pull request #594 from miniduikboot/feature/refine-modular-ac

Change some anticheat checks categories

Add similar AC check to HandleSetName

Move Name checks to NameLimits

Move remaining Color checks to ColorLimits

Some mods add custom colors and still use the standard vanilla SetColor
RPC, which leads to issues with this check.

Merge pull request #593 from Pietrodjaowjao/master

Patch game-breaking exploits

Documentation: Update .NET 7 to .NET 8

Forgot to actually return

gamedata

Merge pull request #591 from NikoCat233/add-broadcast-check-to-most-of-calls

[AntiCheat] Add ValidateBroadCast check to most of the rpc calls

Fix embarrassing bugs

Remove use of SetInfected RPC

Client no longer sends the RPC nor does anything with it when receiving it

Refactor if statements to be more vertically dense

Multiple guard clauses make the file harder to navigate, so put them in
a single guard clause with one validation rule per line. Some statements
in the file were already set up like this, so this commit makes it consistent

Add Broadcast checks to remaining RPC's

Merge pull request #589 from miniduikboot/feature/game-created-event-add-host

Add Host property to IGameCreatedEvent

Add broadcast

Add Host property to IGameCreatedEvent

When a game has just been created, it does not yet have any players in
them, and as a result you can't get the host or its connection id when
this event is called. This makes it annoying to link between this event
and the IGameCreationEvent that came just before it.

Having this link is useful for the usecase of a game code allocation
plugin that allocates codes from a pool, and needs to return codes to
a pool if these codes weren't picked.

Merge pull request #571 from Impostor/bugfix/misc-ac

Miscellaneous Anticheat fixes

Rework Color calculation loop to always terminate

Don't dequeue a color when none are available

Check AC config in SetName/Color checks

Previously this was not done causing these checks to be unconditionally enabled

Allow host-only mods to desync kill timeouts

Allow host-only mods to desync murder roles

Make Impostor's AC modular (#559)

* Add extra configuration options for the anticheat

* Implement option to exempt hosts from AC checks

* Add AntiCheatConfig to InnerNetObject and friends

* Categorize calls in InnerNetObject.Anticheat

* Categorize other calls to ReportCheatAsync

* Update cosmetics AC handling

When Innersloth changed SetHat etc to SetHatStr as part of the cosmicube
update we accidentally created a few dead methods. Restore and update
the code.

* Push up cheat category to ReportCheatAsync

This makes it possible to log failed checks for diagnosis. Also much
cleaner now that we don't have to push AntiCheatConfig everywhere.
Should've done this immediately q.q

* Add back old version of ReportCheatAsync

some plugins used this method. affected plugins should rebuild after the
next impostor release

* Add documentation for modular anticheat

* Address comments

* Sort AntiCheatConfig keys

* Rework the option to allow cheating hosts

This option should actually be tristate: to not break host-only mods we
should disable the anticheat for hosts running host-only mods.

* Document CheatingHostMode

* Split Limits category in Color and Name Limits

Prepare release 1.9.1

Merge pull request #573 from ykundesu/fix/AprilFools2024

fixed AprilFools2024 Deserialize error

Fix compile warning

fixed AprilFools2024 Deserialize error

Merge pull request #569 from Impostor/feature/2024.3.5

Feature/2024.3.5

Run Dumpostor on 2024.3.5

Add new game version for 2024.3.5

Officials don't allow crosplay either, so neither should we