Add support for PackedGameDataTo

Add compat version for 17.3 (#724)

* Add compat version for 17.3

* Add compat version for 17.3.1

Merge pull request #723 from miniduikboot/fix/aof-concdict

Fix race conditions in Game.Data

Rename _allObjectsFast to _allObjects

Done because there is no second datastructure anymore

Switch to concurrent dictionary

PR proposed in #722 doesn't solve all issues, now Nullrefs show up.

Also drop _allObjects as it's never read.

Merge pull request #717 from Impostor/feature/17.2.2

Add 17.2.2 version

Add 17.2.2 version

Only changes are ad related, and officials allow crossplay

Merge pull request #716 from Impostor/feature/17.2-hotfix

Add 17.2 hotfix version

Add 17.2 hotfix version

Merge pull request #715 from Impostor/feature/17.2

Add 17.2 version

Add 17.2 version

https://www.innersloth.com/announcing-among-us-x-ace-attorney-again/

Merge pull request #712 from miniduikboot/feature/aumi-host-chat-cmds

Add support for host chat cmds

Reduce warnings about +25

As Innersloth documented it in their mod info document, assume it's here
to stay, so warn about it less heavily

Implement host chat commands

On January 21 Innersloth released a document that details a few new
features. One of these are host chat commands: any chat message that
starts with "/cmd" is forwarded to the host and to the host only.

Implement this feature in Impostor too for implementation parity.

For more info, see their GitHub page:
https://github.com/Innersloth-LLC/AmongUsModdingInformation

Merge pull request #708 from Impostor/feature/17.1

Bump compatversion to support 17.1

Add another mobile-only hotfix

From their Discord:

Admin has pushed hotfix v17.1.2 for mobile players! minor changes to the ad rewards to the daily cap (subject to future change) and updated the screen to clarify when the daily cap is reset.

Bump compatversion to support 17.1.1

https://www.innersloth.com/ad-rewards-v17-1-1/

Thanks to Niko for reminding me about this

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

Bump compatversion to support 17.1

Merge pull request #706 from NikoCat233/patch-1

Add missing comment for game version 16.1.0

Add missing comment for game version 16.1.0

Merge pull request #705 from Impostor/feature/17.0.1

Bump to 17.0.1

Bump to 17.0.1

Merge pull request #702 from Impostor/feature/17.0.0

Add support for 17.0.0

Run Dumposter on 17.0.0

Add GameOptions V10

Manually added roles to RoleTypes, will clean up later

Bump to 1.10.4

No breaking changes expected

Add version block for 17.0.0

Add trailing comma

Fixes compile warning

Update System.Text.Json

Merge pull request #692 from miniduikboot/doc/https-now-mandatory

Update docs to add that HTTPS is now mandatory

Update FAQ for AU3D

Clarify startup warning

Update documentation for mandatory HTTPS

The new Unity version used by Among Us 16.0.5 disables the
"InsecureHttpOption", which blocks UnityWebRequests via HTTP. Therefore
all Impostor servers should now switch to HTTPS.

https://docs.unity3d.com/2022.3/Documentation/ScriptReference/InsecureHttpOption.html

Thanks to TianMengLucky for reporting this and NikoCat233 for
identifying the cause

Don't clear RoleOptions on update

Among Us always adds a constant amount of roles to this dict, so it
doesn't make sense to clear this dictionary every time.

Furthermore, previously it was possible to trigger a
ConcurrentModificationException when running Deserialize in parallel
with Serialize. This should no longer happen, as new keys are no longer added

Address review comments

Remove AllowNull from PlayerControl.PlayerInfo

It is possible to spawn invalid network objects without an attached
PlayerInfo, and Impostor will throw exceptions everywhere if you happen
to spawn one. Officials do allow spawning objects like this, so allow it
in Impostor for compatibility reasons

Add new cheat category for InvalidObject checks

For some network objects, like PlayerControl, we expect that there is an
PlayerInfo object associated to them. If there isn't, throw an exception
in this AC category

Add Murder/Protect/Exile API without checks

Force(.*)Async will perform the same function as \1Async but without
throwing exceptions. The old API is maintained for backwards
compatibility reasons.

Murder checks are now shared with the main function that handles
CheckMurder, as ProtectPlayerAsync and ExileAsync aren't used in the
code I didn't make a similar change there. Checks were also synced
between both methods

Always store previous color

Because it's only restored in standard SA networking, the stored value
will be ignored in HA. This is preferred to logging when in HA mode

Record and restore player colors

This prevents situations where a player that requested red has a
different color when rejoining because red is the default color of a
PlayerInfo object. This is most noticeable if the host is late with
rejoining, and a lot of players are alreading in WaitingForHost limbo.

When the host finally rejoins, they will not assign Red immediately to
players that requested Red as it is "in use", and instead assign the
next color.

By recording and restoring this is avoided, and players tend to get
their own colors back in most cases.

This is not something that officials do at the moment, but I don't think
this has an impact on mods, as a normal checkcolor/setcolor cycle will
take place as usual.

Merge pull request #690 from Impostor/feature/16.0.5

Add 16.0.5 support

Add 16.0.5

Crossplay with the previous version is not supported, so it gets its own
compatibility group.

Further analysis on the changes is still needed

Merge pull request #683 from Impostor/feature/16.0.0

Add support for 16.0.0

Add note to old game methods

These are no longer used, so warn people who do want to edit them.

I don't foresee these getting removed soon though, as we do want to kick
players on old versions for having an old game version

Add GameTags enum

Can't be generated from game files at the moment unfortunately

Remove GameListingV2

The game ignores unknown fields when deserializing, and as only fields
were added it's acceptable to merge the structs together

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.