Update appveyor.yml

crappveyor

Remove Auth- and AnnouncementsServer sections (#484)

Remove 75% of the question marks of PR #481 (#482)

Add Serilog.Settings.Configuration (#468)

* Add Serilog.Settings.Configuration

* Add docs section for logging

* Elaborate Serilog documentation

Co-authored-by: miniduikboot <mini@duikbo.at>

Merge pull request #483 from js6pak/fix-disconnect-reason

Fix non custom disconnect reasons

Fix non custom disconnect reasons

Allow adding custom tasks on standard maps (#481)

This is needed for mods that add tasks to standard maps, but also mods
that add custom maps: because their custom netobjects don't get
registered Impostor thinks people are still on the old map when people
switch to the custom map after playing a game.

Upgrade to .NET 6 (#479)

* Upgrade to .NET 6

.NET 5 has gone EOL, so move to a supported version.

* Upgrade Docker container as well
* Update docs for .NET 6
* Re-add hack for System.IO.Pipelines
* Bump dependency versions
* Update the deps of the other projects too
* Only declare TargetFramework once
* Dockerfile: also copy over Directory.Build.props
* Cake: pick up target framework automatically
* Upgrade Cake as well

Merge pull request #474 from SubmergedAmongUs/master

Add handling for null ShipStatus

Remove null check for ShipStatus known to not be null

Add handling for null ShipStatus

Merge pull request #471 from Impostor/feature/2022.3.29

Simplify Message07JoinedGameS2CPre20220202

Make GameOptionsData de/serialization less confusing

Do not add ProductUserId/Friendcode on <2.24

Because it is fine to add data to the end of messages the new fields
could be added to Message 1 just fine, but because Message 7 contains a
loop, we need to send a different message to old clients there.

fixup! Add empty ProductUserId and FriendCode to packets

Add empty ProductUserId and FriendCode to packets

They are not sent to the server and no clue how to retrieve them out of band.

Barebones port

Merge pull request #467 from miniduikboot/fix/race-dc-send-all-except

Do not broadcast messages to disconnected clients

Do not broadcast messages to disconnected clients

This method is used when players have disconnected. If multiple players
disconnect at once, this method leads to exceptions.

Merge pull request #464 from miniduikboot/fix/custom-dc-msg

Fix/custom dc msg

Reword obsoletion message

Change DisconnectAsync to use extension instead

Turn DisconnectConnection into an extension

Fixup 608414326ddcbe672b3fd5e71d95530707add6d7

Oops, I accidentally forgot to remove the previous line when adding the
file to my commit

Deprecate JoinGame errors

Starting from version 2021.11.9, join errors no longer exist, instead a
client will show the message "An unknown error disconnected you from the
server". As a workaround, the connection is disconnected entirely when
an error occurs, which allows us to show a custom message.

Merge pull request #465 from miniduikboot/fix/2-28-bump

Bump network version

Merge pull request #460 from miniduikboot/feature/queryplatformids

Add QueryPlatformIds

Add QueryPlatformIds

Some platforms (notable the Microsoft Store version) ask for this before connecting.

Merge pull request #455 from miniduikboot/fix/i18n-nicknames

Fix/i18n nicknames

Merge pull request #458 from miniduikboot/fix/ac-engineer-snapto

AC: Allow engineers to move to a different vent.

Partial revert of cc8d5d2: Readd length check

Oversized nicknames can cause bigger issues, so it is desireable to
still block them. .Length counts codepoints, not bytes, which means
that 10 character Japanese nicknames, which are configurable ingame,
should still be allowed.

AC: Allow engineers to move to a different vent.

Merge pull request #453 from miniduikboot/fix/ga-half-cooldown

AC CheckMurder: Require half of cooldown

Add comment to clarify why this exists

Merge pull request #452 from miniduikboot/fix/host-migrate

Before this commit, when the host of a game disconnects, the InnerPlayerInfo from the player is removed, which causes the new host to disconnect because of a deserialization error. This PR fixes this issue

Remove name length/validity checks from JoinGame

Length and validity are checked also checked by the anticheat. In case
Impostor goes out of sync with the allowed characters again, this can
just be fixed by disabling the Anticheat.

Check for NullOrWhiteSpace is kept because an empty username could be annoying

Sync IsCharAllowed with Innersloth implementation

AC CheckMurder: Require half of cooldown

When the Guardian Angel protects a player, the impostor that just missed
their kill gets back half of their cooldown. Keeping track of whether or
not the player was protected is more difficult due to how the Guardian
Angel is implemented, so instead we halve the minimum time between kill
in order to minimize false positives.

Clamp taskCount to the length of the task list

New tasks cannot be created at this point because we don't know the
underlying tasks. So we try to maintain the tasks we do know of.

Do not destroy PlayerControl if game is starting

This player may still get assigned tasks, which could lead to
deserialization problems.

Mark game as started when RPC SetRole is called

This used to be done when SetInfected was called, but this RPC is no
longer used since 2021.11.9 in favor of this RPC.

Merge pull request #442 from Impostor/2021.11.9

2021.11.9 and 2021.12.14 support

Set RoleType in PlayerInfo.Deserialize

Don't do public methods in interfaces

Readd HandleSet{Pet/Hat/Skin}

Implement cosmetic rpc methods

Document IClient.PlatformSpecificData

Change RoleOptionsData floats to be bytes as they can't ever be floats anyway

Add initial support for 2021.12.14

Add RPC's for CheckProtect and ProtectPlayer

Implement new MurderPlayer logic

In 2021.11.9, the host now uses MurderPlayer instead of impostors.
Impostor now use CheckMurder to request the host to kill a player. This
allows the host to prevent the target of the Guardian Angel from dying.

However, if the impostor that is checking the kill is also the host, AU
internally processes the CheckMurder RPC and sends the MurderPlayer RPC
first, *then* it sends the CheckMurder RPC.

Add platform data to GameList

Fix ServerReplay

Bump to 1.7

Allow Shapeshifters to use RPC Shapeshift

Allow Engineers to vent

Implement Rpc 44: SetRole

Add name/psd/level to JoinGame- and JoinedGameS2C

Two players can now connect to the same lobby

Remove initial state flag from InnerGameData

Implement SetActivePodType

WIP

RIP AnnouncementsServer

[skip ci] Auto deploy github releases

Merge pull request #431 from miniduikboot/feature/2021.6.30

Handle all handshake versions

Language and chat mode api

Improve docs (#435)

Address feedback

Check client version when parsing C2S handshake

2021.6.30 added new fields, which aren't present on old clients.

This commit makes sure players on old versions are disconnected properly
instead of the server erroring internally.

Disable ownership check for BootFromVent

Implement RPC 35: Update System

Implement RPC 34: Boot from vent

Bump to 1.6 because new net version

Add Vent Cleaning task

As these spawn in various rooms under the same task id the name of this
task does not include a room name

Add Airship Lounge/Showers Empty Garbage tasks

Also Calibrate Distributor moved up

Add two extra fields to handshake

Bump client version

Improve error when client version is unsupported (#432)

* Improve error when client version is unsupported

In the default message that was used previously the user is asked to
upgrade his game. This is not always correct, as sometimes the Impostor
server is too old and should be updated instead. Therefore custom
disconnect messages are used that blames either the user or the server
operator, which is hopefully more accurate.

I've chosen not to include the game version that is supported by the
server in this message because the network version rarely matches the
advertised version in the main menu.

* Rewrite versionCompare switch case

Co-authored-by: js6pak <kubastaron@hotmail.com>
Co-authored-by: js6pak <kubastaron@hotmail.com>

Log unsupported version

Merge pull request #430 from Bramux/fix-hat-mismatch

Fix mismatched hats

Correct hat mismatches

Merge pull request #429 from miniduikboot/fix/style

Fix warnings in Impostor.Benchmarks

Use var instead of explicit type

Also prevent future issues

Remove unnecessary imports

And add rule to prevent future issues

Merge pull request #428 from miniduikboot/fix/colortype

Correct ColorType array

Merge pull request #427 from miniduikboot/fix/airship-spawn

Stop cancelling initial Airship spawn SnapTo RPC

Correct ColorType array

When creating this array for the update, I used the wrong array. Oops!
Turns out 5 of these colors aren't actually in the game.

Stop cancelling initial Airship spawn SnapTo RPC

This is used to hide players that are still selecting their spawn
location. If this is cancelled, players are left standing at the
location they were at when the last meeting was called, which is undesirable.

Remove legacy regionInfo patcher

Merge pull request #423 from miniduikboot/feature/2021.6.15

Turn derived properties into getters

There is no significant performance difference: while accessing the
getters is slightly slower, this can be balanced against the cost of
setting the property. As this is also cleaner, switch to getters.

Refactor Polus meeting room spawning logic

Also adjust the constants to the new constants used in game

Stub implementation of SendQuickChat

Proper API will follow later

Remove the table from the Readme

As Among Us is getting a lot of small network-compatible bugfix releases
nowadays, it doesn't seem necessary to maintain the table anymore.
Release notes state clearly which network version is supported and there
is limited support for old versions anyway.

Simplify the setup instructions

The patcher has not been updated to support the new JSON format and is
less necessary now that JSON is used for the region file format.

The website also now has deeplinking support, so we point Android/iOS
people to the website now too as that is more convenient compared to the
previous methods.

Meeting changes

Announcements protocol changes