From ea1ea5cee7cdec4a5326e65e8e7316eae2d4195d Mon Sep 17 00:00:00 2001 From: Timothy Schenk Date: Wed, 1 Nov 2023 18:18:27 +0100 Subject: [PATCH] chore: switch to dotnet foundation editorconfig preset --- .editorconfig | 838 +++++++++--------- Benchmarks/BinaryConversionBenchmarks.cs | 16 +- Continuity.sln | 9 - Server/AuthSession.cs | 34 +- Server/ChannelSession.cs | 32 +- Server/Consumers/PacketConsumer.cs | 6 +- Server/DB/WonderkingContext.cs | 12 +- Server/LoggerMessages/PacketLoggerMessages.cs | 1 - .../PacketHandlers/ChannelSelectionHandler.cs | 14 +- Server/PacketHandlers/LoginHandler.cs | 28 +- Server/Program.cs | 4 +- Server/Server.csproj | 8 +- Server/Services/PacketDistributorService.cs | 52 +- Server/Services/WonderkingAuthServer.cs | 20 +- build/.editorconfig | 11 - build/Build.cs | 116 --- build/Configuration.cs | 11 - build/Directory.Build.props | 8 - build/Directory.Build.targets | 8 - build/build.csproj | 23 - build/build.csproj.DotSettings | 28 - 21 files changed, 535 insertions(+), 744 deletions(-) delete mode 100644 build/.editorconfig delete mode 100644 build/Build.cs delete mode 100644 build/Configuration.cs delete mode 100644 build/Directory.Build.props delete mode 100644 build/Directory.Build.targets delete mode 100644 build/build.csproj delete mode 100644 build/build.csproj.DotSettings diff --git a/.editorconfig b/.editorconfig index 4d9252b..3c8f65b 100644 --- a/.editorconfig +++ b/.editorconfig @@ -1,453 +1,461 @@ -# Version: 4.1.1 (Using https://semver.org/) -# Updated: 2022-05-23 -# See https://github.com/RehanSaeed/EditorConfig/releases for release notes. -# See https://github.com/RehanSaeed/EditorConfig for updates to this file. -# See http://EditorConfig.org for more information about .editorconfig files. +; EditorConfig to support per-solution formatting. +; Use the EditorConfig VS add-in to make this work. +; http://editorconfig.org/ +; +; Here are some resources for what's supported for .NET/C# +; https://kent-boogaart.com/blog/editorconfig-reference-for-c-developers +; https://learn.microsoft.com/en-us/visualstudio/ide/editorconfig-code-style-settings-reference +; +; Be **careful** editing this because some of the rules don't support adding a severity level +; For instance if you change to `dotnet_sort_system_directives_first = true:warning` (adding `:warning`) +; then the rule will be silently ignored. -########################################## -# Common Settings -########################################## - -# This file is the top-most EditorConfig file +; This is the default for the codeline. root = true -# All Files [*] -charset = utf-8 indent_style = space -indent_size = 4 -insert_final_newline = true +charset = utf-8 trim_trailing_whitespace = true +insert_final_newline = true -########################################## -# File Extension Settings -########################################## - -# Visual Studio Solution Files -[*.sln] -indent_style = tab - -# Visual Studio XML Project Files -[*.{csproj,vbproj,vcxproj.filters,proj,projitems,shproj}] -indent_size = 2 - -# XML Configuration Files -[*.{xml,config,props,targets,nuspec,resx,ruleset,vsixmanifest,vsct}] -indent_size = 2 - -# JSON Files -[*.{json,json5,webmanifest}] -indent_size = 2 - -# YAML Files -[*.{yml,yaml}] -indent_size = 2 - -# Markdown Files -[*.{md,mdx}] -trim_trailing_whitespace = false - -# Web Files -[*.{htm,html,js,jsm,ts,tsx,cjs,cts,ctsx,mjs,mts,mtsx,css,sass,scss,less,pcss,svg,vue}] -indent_size = 2 - -# Batch Files -[*.{cmd,bat}] -end_of_line = crlf - -# Bash Files -[*.sh] -end_of_line = lf - -# Makefiles -[Makefile] -indent_style = tab - -########################################## -# Default .NET Code Style Severities -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/configuration-options#scope -########################################## - -[*.{cs,csx,cake,vb,vbx}] -# Default Severity for all .NET Code Style rules below -dotnet_analyzer_diagnostic.severity = warning - -########################################## -# Language Rules -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/language-rules -########################################## - -# .NET Style Rules -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/language-rules#net-style-rules -[*.{cs,csx,cake,vb,vbx}] -# "this." and "Me." qualifiers -dotnet_style_qualification_for_field = true:warning -dotnet_style_qualification_for_property = true:warning -dotnet_style_qualification_for_method = true:warning -dotnet_style_qualification_for_event = true:warning -# Language keywords instead of framework type names for type references -dotnet_style_predefined_type_for_locals_parameters_members = true:warning -dotnet_style_predefined_type_for_member_access = true:warning -# Modifier preferences -dotnet_style_require_accessibility_modifiers = always:warning -csharp_preferred_modifier_order = public, private, protected, internal, static, extern, new, virtual, abstract, sealed, override, readonly, unsafe, volatile, async:warning -visual_basic_preferred_modifier_order = Partial, Default, Private, Protected, Public, Friend, NotOverridable, Overridable, MustOverride, Overloads, Overrides, MustInherit, NotInheritable, Static, Shared, Shadows, ReadOnly, WriteOnly, Dim, Const, WithEvents, Widening, Narrowing, Custom, Async:warning -dotnet_style_readonly_field = true:warning -# Parentheses preferences -dotnet_style_parentheses_in_arithmetic_binary_operators = always_for_clarity:warning -dotnet_style_parentheses_in_relational_binary_operators = always_for_clarity:warning -dotnet_style_parentheses_in_other_binary_operators = always_for_clarity:warning -dotnet_style_parentheses_in_other_operators = never_if_unnecessary:warning -# Expression-level preferences -dotnet_style_object_initializer = true:warning -dotnet_style_collection_initializer = true:warning -dotnet_style_explicit_tuple_names = true:warning -dotnet_style_prefer_inferred_tuple_names = true:warning -dotnet_style_prefer_inferred_anonymous_type_member_names = true:warning -dotnet_style_prefer_auto_properties = true:warning -dotnet_style_prefer_conditional_expression_over_assignment = false:suggestion -dotnet_diagnostic.IDE0045.severity = suggestion -dotnet_style_prefer_conditional_expression_over_return = false:suggestion -dotnet_diagnostic.IDE0046.severity = suggestion -dotnet_style_prefer_compound_assignment = true:warning -dotnet_style_prefer_simplified_interpolation = true:warning -dotnet_style_prefer_simplified_boolean_expressions = true:warning -# Null-checking preferences -dotnet_style_coalesce_expression = true:warning -dotnet_style_null_propagation = true:warning -dotnet_style_prefer_is_null_check_over_reference_equality_method = true:warning -# File header preferences -# file_header_template = \n© PROJECT-AUTHOR\n -# If you use StyleCop, you'll need to disable SA1636: File header copyright text should match. -# dotnet_diagnostic.SA1636.severity = none -# Undocumented -dotnet_style_operator_placement_when_wrapping = end_of_line:warning -csharp_style_prefer_null_check_over_type_check = true:warning - -# C# Style Rules -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/language-rules#c-style-rules -[*.{cs,csx,cake}] -# 'var' preferences -csharp_style_var_for_built_in_types = true:warning -csharp_style_var_when_type_is_apparent = true:warning -csharp_style_var_elsewhere = true:warning -# Expression-bodied members -csharp_style_expression_bodied_methods = true:warning -csharp_style_expression_bodied_constructors = true:warning -csharp_style_expression_bodied_operators = true:warning -csharp_style_expression_bodied_properties = true:warning -csharp_style_expression_bodied_indexers = true:warning -csharp_style_expression_bodied_accessors = true:warning -csharp_style_expression_bodied_lambdas = true:warning -csharp_style_expression_bodied_local_functions = true:warning -# Pattern matching preferences -csharp_style_pattern_matching_over_is_with_cast_check = true:warning -csharp_style_pattern_matching_over_as_with_null_check = true:warning -csharp_style_prefer_switch_expression = true:warning -csharp_style_prefer_pattern_matching = true:warning -csharp_style_prefer_not_pattern = true:warning -# Expression-level preferences -csharp_style_inlined_variable_declaration = true:warning -csharp_prefer_simple_default_expression = true:warning -csharp_style_pattern_local_over_anonymous_function = true:warning -csharp_style_deconstructed_variable_declaration = true:warning -csharp_style_prefer_index_operator = true:warning -csharp_style_prefer_range_operator = true:warning -csharp_style_implicit_object_creation_when_type_is_apparent = true:warning -# "Null" checking preferences -csharp_style_throw_expression = true:warning -csharp_style_conditional_delegate_call = true:warning -# Code block preferences -csharp_prefer_braces = true:warning -csharp_prefer_simple_using_statement = true:suggestion -dotnet_diagnostic.IDE0063.severity = suggestion -# 'using' directive preferences -csharp_using_directive_placement = inside_namespace:warning -# Modifier preferences -csharp_prefer_static_local_function = true:warning - -########################################## -# Unnecessary Code Rules -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/unnecessary-code-rules -########################################## - -# .NET Unnecessary code rules -[*.{cs,csx,cake,vb,vbx}] -dotnet_code_quality_unused_parameters = all:warning -dotnet_remove_unnecessary_suppression_exclusions = none:warning - -# C# Unnecessary code rules -[*.{cs,csx,cake}] -csharp_style_unused_value_expression_statement_preference = discard_variable:suggestion -dotnet_diagnostic.IDE0058.severity = suggestion -csharp_style_unused_value_assignment_preference = discard_variable:suggestion -dotnet_diagnostic.IDE0059.severity = suggestion - -########################################## -# Formatting Rules -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules -########################################## - -# .NET formatting rules -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules#net-formatting-rules -[*.{cs,csx,cake,vb,vbx}] -# Organize using directives +[*.cs] +indent_size = 4 dotnet_sort_system_directives_first = true -dotnet_separate_import_directive_groups = false -# Dotnet namespace options -dotnet_style_namespace_match_folder = true:suggestion -dotnet_diagnostic.IDE0130.severity = suggestion -# C# formatting rules -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules#c-formatting-rules -[*.{cs,csx,cake}] -# Newline options -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules#new-line-options +# Don't use this. qualifier +dotnet_style_qualification_for_field = false:suggestion +dotnet_style_qualification_for_property = false:suggestion + +# use int x = .. over Int32 +dotnet_style_predefined_type_for_locals_parameters_members = true:suggestion + +# use int.MaxValue over Int32.MaxValue +dotnet_style_predefined_type_for_member_access = true:suggestion + +# Require var all the time. +csharp_style_var_for_built_in_types = true:suggestion +csharp_style_var_when_type_is_apparent = true:suggestion +csharp_style_var_elsewhere = true:suggestion + +# Disallow throw expressions. +csharp_style_throw_expression = false:suggestion + +# Newline settings csharp_new_line_before_open_brace = all csharp_new_line_before_else = true csharp_new_line_before_catch = true csharp_new_line_before_finally = true csharp_new_line_before_members_in_object_initializers = true csharp_new_line_before_members_in_anonymous_types = true -csharp_new_line_between_query_expression_clauses = true -# Indentation options -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules#indentation-options -csharp_indent_case_contents = true -csharp_indent_switch_labels = true -csharp_indent_labels = no_change -csharp_indent_block_contents = true -csharp_indent_braces = false -csharp_indent_case_contents_when_block = false -# Spacing options -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules#spacing-options -csharp_space_after_cast = false -csharp_space_after_keywords_in_control_flow_statements = true -csharp_space_between_parentheses = false -csharp_space_before_colon_in_inheritance_clause = true -csharp_space_after_colon_in_inheritance_clause = true -csharp_space_around_binary_operators = before_and_after -csharp_space_between_method_declaration_parameter_list_parentheses = false -csharp_space_between_method_declaration_empty_parameter_list_parentheses = false -csharp_space_between_method_declaration_name_and_open_parenthesis = false -csharp_space_between_method_call_parameter_list_parentheses = false -csharp_space_between_method_call_empty_parameter_list_parentheses = false -csharp_space_between_method_call_name_and_opening_parenthesis = false -csharp_space_after_comma = true -csharp_space_before_comma = false -csharp_space_after_dot = false -csharp_space_before_dot = false -csharp_space_after_semicolon_in_for_statement = true -csharp_space_before_semicolon_in_for_statement = false -csharp_space_around_declaration_statements = false -csharp_space_before_open_square_brackets = false -csharp_space_between_empty_square_brackets = false -csharp_space_between_square_brackets = false -# Wrap options -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules#wrap-options -csharp_preserve_single_line_statements = false -csharp_preserve_single_line_blocks = true -# Namespace options -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules#namespace-options -csharp_style_namespace_declarations = file_scoped:warning -########################################## -# .NET Naming Rules -# https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/naming-rules -########################################## +# Namespace settings +csharp_style_namespace_declarations = file_scoped -[*.{cs,csx,cake,vb,vbx}] +# Brace settings +csharp_prefer_braces = true # Prefer curly braces even for one line of code -########################################## -# Styles -########################################## - -# camel_case_style - Define the camelCase style -dotnet_naming_style.camel_case_style.capitalization = camel_case -# pascal_case_style - Define the PascalCase style +# name all constant fields using PascalCase +dotnet_naming_rule.constant_fields_should_be_pascal_case.severity = suggestion +dotnet_naming_rule.constant_fields_should_be_pascal_case.symbols = constant_fields +dotnet_naming_rule.constant_fields_should_be_pascal_case.style = pascal_case_style +dotnet_naming_symbols.constant_fields.applicable_kinds = field +dotnet_naming_symbols.constant_fields.required_modifiers = const dotnet_naming_style.pascal_case_style.capitalization = pascal_case -# first_upper_style - The first character must start with an upper-case character -dotnet_naming_style.first_upper_style.capitalization = first_word_upper -# prefix_interface_with_i_style - Interfaces must be PascalCase and the first character of an interface must be an 'I' -dotnet_naming_style.prefix_interface_with_i_style.capitalization = pascal_case -dotnet_naming_style.prefix_interface_with_i_style.required_prefix = I -# prefix_type_parameters_with_t_style - Generic Type Parameters must be PascalCase and the first character must be a 'T' -dotnet_naming_style.prefix_type_parameters_with_t_style.capitalization = pascal_case -dotnet_naming_style.prefix_type_parameters_with_t_style.required_prefix = T -# disallowed_style - Anything that has this style applied is marked as disallowed -dotnet_naming_style.disallowed_style.capitalization = pascal_case -dotnet_naming_style.disallowed_style.required_prefix = ____RULE_VIOLATION____ -dotnet_naming_style.disallowed_style.required_suffix = ____RULE_VIOLATION____ -# internal_error_style - This style should never occur... if it does, it indicates a bug in file or in the parser using the file -dotnet_naming_style.internal_error_style.capitalization = pascal_case -dotnet_naming_style.internal_error_style.required_prefix = ____INTERNAL_ERROR____ -dotnet_naming_style.internal_error_style.required_suffix = ____INTERNAL_ERROR____ -########################################## -# .NET Design Guideline Field Naming Rules -# Naming rules for fields follow the .NET Framework design guidelines -# https://docs.microsoft.com/dotnet/standard/design-guidelines/index -########################################## +# internal and private fields should be _camelCase +dotnet_naming_rule.camel_case_for_private_internal_fields.severity = suggestion +dotnet_naming_rule.camel_case_for_private_internal_fields.symbols = private_internal_fields +dotnet_naming_rule.camel_case_for_private_internal_fields.style = camel_case_underscore_style +dotnet_naming_symbols.private_internal_fields.applicable_kinds = field +dotnet_naming_symbols.private_internal_fields.applicable_accessibilities = private, internal +dotnet_naming_style.camel_case_underscore_style.required_prefix = _ +dotnet_naming_style.camel_case_underscore_style.capitalization = camel_case -# All public/protected/protected_internal constant fields must be PascalCase -# https://docs.microsoft.com/dotnet/standard/design-guidelines/field -dotnet_naming_symbols.public_protected_constant_fields_group.applicable_accessibilities = public, protected, protected_internal -dotnet_naming_symbols.public_protected_constant_fields_group.required_modifiers = const -dotnet_naming_symbols.public_protected_constant_fields_group.applicable_kinds = field -dotnet_naming_rule.public_protected_constant_fields_must_be_pascal_case_rule.symbols = public_protected_constant_fields_group -dotnet_naming_rule.public_protected_constant_fields_must_be_pascal_case_rule.style = pascal_case_style -dotnet_naming_rule.public_protected_constant_fields_must_be_pascal_case_rule.severity = warning +[*.{xml,config,*proj,nuspec,props,resx,targets,yml,tasks}] +indent_size = 2 -# All public/protected/protected_internal static readonly fields must be PascalCase -# https://docs.microsoft.com/dotnet/standard/design-guidelines/field -dotnet_naming_symbols.public_protected_static_readonly_fields_group.applicable_accessibilities = public, protected, protected_internal -dotnet_naming_symbols.public_protected_static_readonly_fields_group.required_modifiers = static, readonly -dotnet_naming_symbols.public_protected_static_readonly_fields_group.applicable_kinds = field -dotnet_naming_rule.public_protected_static_readonly_fields_must_be_pascal_case_rule.symbols = public_protected_static_readonly_fields_group -dotnet_naming_rule.public_protected_static_readonly_fields_must_be_pascal_case_rule.style = pascal_case_style -dotnet_naming_rule.public_protected_static_readonly_fields_must_be_pascal_case_rule.severity = warning +# Xml config files +[*.{props,targets,ruleset,config,nuspec,resx,vsixmanifest,vsct}] +indent_size = 2 -# No other public/protected/protected_internal fields are allowed -# https://docs.microsoft.com/dotnet/standard/design-guidelines/field -dotnet_naming_symbols.other_public_protected_fields_group.applicable_accessibilities = public, protected, protected_internal -dotnet_naming_symbols.other_public_protected_fields_group.applicable_kinds = field -dotnet_naming_rule.other_public_protected_fields_disallowed_rule.symbols = other_public_protected_fields_group -dotnet_naming_rule.other_public_protected_fields_disallowed_rule.style = disallowed_style -dotnet_naming_rule.other_public_protected_fields_disallowed_rule.severity = error +[*.json] +indent_size = 2 -########################################## -# StyleCop Field Naming Rules -# Naming rules for fields follow the StyleCop analyzers -# This does not override any rules using disallowed_style above -# https://github.com/DotNetAnalyzers/StyleCopAnalyzers -########################################## +[*.{ps1,psm1}] +indent_size = 4 -# All constant fields must be PascalCase -# https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1303.md -dotnet_naming_symbols.stylecop_constant_fields_group.applicable_accessibilities = public, internal, protected_internal, protected, private_protected, private -dotnet_naming_symbols.stylecop_constant_fields_group.required_modifiers = const -dotnet_naming_symbols.stylecop_constant_fields_group.applicable_kinds = field -dotnet_naming_rule.stylecop_constant_fields_must_be_pascal_case_rule.symbols = stylecop_constant_fields_group -dotnet_naming_rule.stylecop_constant_fields_must_be_pascal_case_rule.style = pascal_case_style -dotnet_naming_rule.stylecop_constant_fields_must_be_pascal_case_rule.severity = warning +[*.sh] +indent_size = 4 +end_of_line = lf -# All static readonly fields must be PascalCase -# https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1311.md -dotnet_naming_symbols.stylecop_static_readonly_fields_group.applicable_accessibilities = public, internal, protected_internal, protected, private_protected, private -dotnet_naming_symbols.stylecop_static_readonly_fields_group.required_modifiers = static, readonly -dotnet_naming_symbols.stylecop_static_readonly_fields_group.applicable_kinds = field -dotnet_naming_rule.stylecop_static_readonly_fields_must_be_pascal_case_rule.symbols = stylecop_static_readonly_fields_group -dotnet_naming_rule.stylecop_static_readonly_fields_must_be_pascal_case_rule.style = pascal_case_style -dotnet_naming_rule.stylecop_static_readonly_fields_must_be_pascal_case_rule.severity = warning +[*.{razor,cshtml}] +charset = utf-8-bom -# No non-private instance fields are allowed -# https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1401.md -dotnet_naming_symbols.stylecop_fields_must_be_private_group.applicable_accessibilities = public, internal, protected_internal, protected, private_protected -dotnet_naming_symbols.stylecop_fields_must_be_private_group.applicable_kinds = field -dotnet_naming_rule.stylecop_instance_fields_must_be_private_rule.symbols = stylecop_fields_must_be_private_group -dotnet_naming_rule.stylecop_instance_fields_must_be_private_rule.style = disallowed_style -dotnet_naming_rule.stylecop_instance_fields_must_be_private_rule.severity = error +[*.{cs,vb}] -# Private fields must be camelCase -# https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1306.md -dotnet_naming_symbols.stylecop_private_fields_group.applicable_accessibilities = private -dotnet_naming_symbols.stylecop_private_fields_group.applicable_kinds = field -dotnet_naming_rule.stylecop_private_fields_must_be_camel_case_rule.symbols = stylecop_private_fields_group -dotnet_naming_rule.stylecop_private_fields_must_be_camel_case_rule.style = camel_case_style -dotnet_naming_rule.stylecop_private_fields_must_be_camel_case_rule.severity = warning +# SYSLIB1054: Use 'LibraryImportAttribute' instead of 'DllImportAttribute' to generate P/Invoke marshalling code at compile time +dotnet_diagnostic.SYSLIB1054.severity = warning -# Local variables must be camelCase -# https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1312.md -dotnet_naming_symbols.stylecop_local_fields_group.applicable_accessibilities = local -dotnet_naming_symbols.stylecop_local_fields_group.applicable_kinds = local -dotnet_naming_rule.stylecop_local_fields_must_be_camel_case_rule.symbols = stylecop_local_fields_group -dotnet_naming_rule.stylecop_local_fields_must_be_camel_case_rule.style = camel_case_style -dotnet_naming_rule.stylecop_local_fields_must_be_camel_case_rule.severity = silent +# CA1018: Mark attributes with AttributeUsageAttribute +dotnet_diagnostic.CA1018.severity = warning -# This rule should never fire. However, it's included for at least two purposes: -# First, it helps to understand, reason about, and root-case certain types of issues, such as bugs in .editorconfig parsers. -# Second, it helps to raise immediate awareness if a new field type is added (as occurred recently in C#). -dotnet_naming_symbols.sanity_check_uncovered_field_case_group.applicable_accessibilities = * -dotnet_naming_symbols.sanity_check_uncovered_field_case_group.applicable_kinds = field -dotnet_naming_rule.sanity_check_uncovered_field_case_rule.symbols = sanity_check_uncovered_field_case_group -dotnet_naming_rule.sanity_check_uncovered_field_case_rule.style = internal_error_style -dotnet_naming_rule.sanity_check_uncovered_field_case_rule.severity = error +# CA1047: Do not declare protected member in sealed type +dotnet_diagnostic.CA1047.severity = warning +# CA1305: Specify IFormatProvider +dotnet_diagnostic.CA1305.severity = warning -########################################## -# Other Naming Rules -########################################## +# CA1507: Use nameof to express symbol names +dotnet_diagnostic.CA1507.severity = warning -# All of the following must be PascalCase: -# - Namespaces -# https://docs.microsoft.com/dotnet/standard/design-guidelines/names-of-namespaces -# https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1300.md -# - Classes and Enumerations -# https://docs.microsoft.com/dotnet/standard/design-guidelines/names-of-classes-structs-and-interfaces -# https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1300.md -# - Delegates -# https://docs.microsoft.com/dotnet/standard/design-guidelines/names-of-classes-structs-and-interfaces#names-of-common-types -# - Constructors, Properties, Events, Methods -# https://docs.microsoft.com/dotnet/standard/design-guidelines/names-of-type-members -dotnet_naming_symbols.element_group.applicable_kinds = namespace, class, enum, struct, delegate, event, method, property -dotnet_naming_rule.element_rule.symbols = element_group -dotnet_naming_rule.element_rule.style = pascal_case_style -dotnet_naming_rule.element_rule.severity = warning +# CA1510: Use ArgumentNullException throw helper +dotnet_diagnostic.CA1510.severity = warning -# Interfaces use PascalCase and are prefixed with uppercase 'I' -# https://docs.microsoft.com/dotnet/standard/design-guidelines/names-of-classes-structs-and-interfaces -dotnet_naming_symbols.interface_group.applicable_kinds = interface -dotnet_naming_rule.interface_rule.symbols = interface_group -dotnet_naming_rule.interface_rule.style = prefix_interface_with_i_style -dotnet_naming_rule.interface_rule.severity = warning +# CA1511: Use ArgumentException throw helper +dotnet_diagnostic.CA1511.severity = warning -# Generics Type Parameters use PascalCase and are prefixed with uppercase 'T' -# https://docs.microsoft.com/dotnet/standard/design-guidelines/names-of-classes-structs-and-interfaces -dotnet_naming_symbols.type_parameter_group.applicable_kinds = type_parameter -dotnet_naming_rule.type_parameter_rule.symbols = type_parameter_group -dotnet_naming_rule.type_parameter_rule.style = prefix_type_parameters_with_t_style -dotnet_naming_rule.type_parameter_rule.severity = warning +# CA1512: Use ArgumentOutOfRangeException throw helper +dotnet_diagnostic.CA1512.severity = warning -# Function parameters use camelCase -# https://docs.microsoft.com/dotnet/standard/design-guidelines/naming-parameters -dotnet_naming_symbols.parameters_group.applicable_kinds = parameter -dotnet_naming_rule.parameters_rule.symbols = parameters_group -dotnet_naming_rule.parameters_rule.style = camel_case_style -dotnet_naming_rule.parameters_rule.severity = warning +# CA1513: Use ObjectDisposedException throw helper +dotnet_diagnostic.CA1513.severity = warning -# Disable warnings for using LoggerMessage delegates -dotnet_diagnostic.CA1848.severity = none +# CA1725: Parameter names should match base declaration +dotnet_diagnostic.CA1725.severity = suggestion -########################################## -# License -########################################## -# The following applies as to the .editorconfig file ONLY, and is -# included below for reference, per the requirements of the license -# corresponding to this .editorconfig file. -# See: https://github.com/RehanSaeed/EditorConfig -# -# MIT License -# -# Copyright (c) 2017-2019 Muhammad Rehan Saeed -# Copyright (c) 2019 Henry Gabryjelski -# -# Permission is hereby granted, free of charge, to any -# person obtaining a copy of this software and associated -# documentation files (the "Software"), to deal in the -# Software without restriction, including without limitation -# the rights to use, copy, modify, merge, publish, distribute, -# sublicense, and/or sell copies of the Software, and to permit -# persons to whom the Software is furnished to do so, subject -# to the following conditions: -# -# The above copyright notice and this permission notice shall be -# included in all copies or substantial portions of the Software. -# -# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, -# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES -# OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND -# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT -# HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, -# WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING -# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR -# OTHER DEALINGS IN THE SOFTWARE. -########################################## \ No newline at end of file +# CA1802: Use literals where appropriate +dotnet_diagnostic.CA1802.severity = warning + +# CA1805: Do not initialize unnecessarily +dotnet_diagnostic.CA1805.severity = warning + +# CA1810: Do not initialize unnecessarily +dotnet_diagnostic.CA1810.severity = warning + +# CA1821: Remove empty Finalizers +dotnet_diagnostic.CA1821.severity = warning + +# CA1822: Make member static +dotnet_diagnostic.CA1822.severity = warning +dotnet_code_quality.CA1822.api_surface = private, internal + +# CA1823: Avoid unused private fields +dotnet_diagnostic.CA1823.severity = warning + +# CA1825: Avoid zero-length array allocations +dotnet_diagnostic.CA1825.severity = warning + +# CA1826: Do not use Enumerable methods on indexable collections. Instead use the collection directly +dotnet_diagnostic.CA1826.severity = warning + +# CA1827: Do not use Count() or LongCount() when Any() can be used +dotnet_diagnostic.CA1827.severity = warning + +# CA1828: Do not use CountAsync() or LongCountAsync() when AnyAsync() can be used +dotnet_diagnostic.CA1828.severity = warning + +# CA1829: Use Length/Count property instead of Count() when available +dotnet_diagnostic.CA1829.severity = warning + +# CA1830: Prefer strongly-typed Append and Insert method overloads on StringBuilder +dotnet_diagnostic.CA1830.severity = warning + +# CA1831: Use AsSpan or AsMemory instead of Range-based indexers when appropriate +dotnet_diagnostic.CA1831.severity = warning + +# CA1832: Use AsSpan or AsMemory instead of Range-based indexers when appropriate +dotnet_diagnostic.CA1832.severity = warning + +# CA1833: Use AsSpan or AsMemory instead of Range-based indexers when appropriate +dotnet_diagnostic.CA1833.severity = warning + +# CA1834: Consider using 'StringBuilder.Append(char)' when applicable +dotnet_diagnostic.CA1834.severity = warning + +# CA1835: Prefer the 'Memory'-based overloads for 'ReadAsync' and 'WriteAsync' +dotnet_diagnostic.CA1835.severity = warning + +# CA1836: Prefer IsEmpty over Count +dotnet_diagnostic.CA1836.severity = warning + +# CA1837: Use 'Environment.ProcessId' +dotnet_diagnostic.CA1837.severity = warning + +# CA1838: Avoid 'StringBuilder' parameters for P/Invokes +dotnet_diagnostic.CA1838.severity = warning + +# CA1839: Use 'Environment.ProcessPath' +dotnet_diagnostic.CA1839.severity = warning + +# CA1840: Use 'Environment.CurrentManagedThreadId' +dotnet_diagnostic.CA1840.severity = warning + +# CA1841: Prefer Dictionary.Contains methods +dotnet_diagnostic.CA1841.severity = warning + +# CA1842: Do not use 'WhenAll' with a single task +dotnet_diagnostic.CA1842.severity = warning + +# CA1843: Do not use 'WaitAll' with a single task +dotnet_diagnostic.CA1843.severity = warning + +# CA1844: Provide memory-based overrides of async methods when subclassing 'Stream' +dotnet_diagnostic.CA1844.severity = warning + +# CA1845: Use span-based 'string.Concat' +dotnet_diagnostic.CA1845.severity = warning + +# CA1846: Prefer AsSpan over Substring +dotnet_diagnostic.CA1846.severity = warning + +# CA1847: Use string.Contains(char) instead of string.Contains(string) with single characters +dotnet_diagnostic.CA1847.severity = warning + +# CA1852: Seal internal types +dotnet_diagnostic.CA1852.severity = warning + +# CA1854: Prefer the IDictionary.TryGetValue(TKey, out TValue) method +dotnet_diagnostic.CA1854.severity = warning + +# CA1855: Prefer 'Clear' over 'Fill' +dotnet_diagnostic.CA1855.severity = warning + +# CA1856: Incorrect usage of ConstantExpected attribute +dotnet_diagnostic.CA1856.severity = error + +# CA1857: A constant is expected for the parameter +dotnet_diagnostic.CA1857.severity = warning + +# CA1858: Use 'StartsWith' instead of 'IndexOf' +dotnet_diagnostic.CA1858.severity = warning + +# CA2007: Consider calling ConfigureAwait on the awaited task +dotnet_diagnostic.CA2007.severity = warning + +# CA2008: Do not create tasks without passing a TaskScheduler +dotnet_diagnostic.CA2008.severity = warning + +# CA2009: Do not call ToImmutableCollection on an ImmutableCollection value +dotnet_diagnostic.CA2009.severity = warning + +# CA2011: Avoid infinite recursion +dotnet_diagnostic.CA2011.severity = warning + +# CA2012: Use ValueTask correctly +dotnet_diagnostic.CA2012.severity = warning + +# CA2013: Do not use ReferenceEquals with value types +dotnet_diagnostic.CA2013.severity = warning + +# CA2014: Do not use stackalloc in loops. +dotnet_diagnostic.CA2014.severity = warning + +# CA2016: Forward the 'CancellationToken' parameter to methods that take one +dotnet_diagnostic.CA2016.severity = warning + +# CA2200: Rethrow to preserve stack details +dotnet_diagnostic.CA2200.severity = warning + +# CA2201: Do not raise reserved exception types +dotnet_diagnostic.CA2201.severity = warning + +# CA2208: Instantiate argument exceptions correctly +dotnet_diagnostic.CA2208.severity = warning + +# CA2245: Do not assign a property to itself +dotnet_diagnostic.CA2245.severity = warning + +# CA2246: Assigning symbol and its member in the same statement +dotnet_diagnostic.CA2246.severity = warning + +# CA2249: Use string.Contains instead of string.IndexOf to improve readability. +dotnet_diagnostic.CA2249.severity = warning + +# IDE0005: Remove unnecessary usings +dotnet_diagnostic.IDE0005.severity = warning + +# IDE0011: Curly braces to surround blocks of code +dotnet_diagnostic.IDE0011.severity = warning + +# IDE0020: Use pattern matching to avoid is check followed by a cast (with variable) +dotnet_diagnostic.IDE0020.severity = warning + +# IDE0029: Use coalesce expression (non-nullable types) +dotnet_diagnostic.IDE0029.severity = warning + +# IDE0030: Use coalesce expression (nullable types) +dotnet_diagnostic.IDE0030.severity = warning + +# IDE0031: Use null propagation +dotnet_diagnostic.IDE0031.severity = warning + +# IDE0035: Remove unreachable code +dotnet_diagnostic.IDE0035.severity = warning + +# IDE0036: Order modifiers +csharp_preferred_modifier_order = public,private,protected,internal,static,extern,new,virtual,abstract,sealed,override,readonly,unsafe,volatile,async:suggestion +dotnet_diagnostic.IDE0036.severity = warning + +# IDE0038: Use pattern matching to avoid is check followed by a cast (without variable) +dotnet_diagnostic.IDE0038.severity = warning + +# IDE0043: Format string contains invalid placeholder +dotnet_diagnostic.IDE0043.severity = warning + +# IDE0044: Make field readonly +dotnet_diagnostic.IDE0044.severity = warning + +# IDE0051: Remove unused private members +dotnet_diagnostic.IDE0051.severity = warning + +# IDE0055: All formatting rules +dotnet_diagnostic.IDE0055.severity = suggestion + +# IDE0059: Unnecessary assignment to a value +dotnet_diagnostic.IDE0059.severity = warning + +# IDE0060: Remove unused parameter +dotnet_code_quality_unused_parameters = non_public +dotnet_diagnostic.IDE0060.severity = warning + +# IDE0062: Make local function static +dotnet_diagnostic.IDE0062.severity = warning + +# IDE0073: File header +dotnet_diagnostic.IDE0073.severity = warning +# Not a dotnet foundation project +#file_header_template = Licensed to the .NET Foundation under one or more agreements.\nThe .NET Foundation licenses this file to you under the MIT license. + +# IDE0161: Convert to file-scoped namespace +dotnet_diagnostic.IDE0161.severity = warning + +# IDE0200: Lambda expression can be removed +dotnet_diagnostic.IDE0200.severity = warning + +# IDE2000: Disallow multiple blank lines +dotnet_style_allow_multiple_blank_lines_experimental = false +dotnet_diagnostic.IDE2000.severity = warning + +[{eng/tools/**.cs,**/{test,testassets,samples,Samples,perf,benchmarkapps,scripts,stress}/**.cs,src/Hosting/Server.IntegrationTesting/**.cs,src/Servers/IIS/IntegrationTesting.IIS/**.cs,src/Shared/Http2cat/**.cs,src/Testing/**.cs}] +# CA1018: Mark attributes with AttributeUsageAttribute +dotnet_diagnostic.CA1018.severity = suggestion +# CA1507: Use nameof to express symbol names +dotnet_diagnostic.CA1507.severity = suggestion +# CA1510: Use ArgumentNullException throw helper +dotnet_diagnostic.CA1510.severity = suggestion +# CA1511: Use ArgumentException throw helper +dotnet_diagnostic.CA1511.severity = suggestion +# CA1512: Use ArgumentOutOfRangeException throw helper +dotnet_diagnostic.CA1512.severity = suggestion +# CA1513: Use ObjectDisposedException throw helper +dotnet_diagnostic.CA1513.severity = suggestion +# CA1802: Use literals where appropriate +dotnet_diagnostic.CA1802.severity = suggestion +# CA1805: Do not initialize unnecessarily +dotnet_diagnostic.CA1805.severity = suggestion +# CA1810: Do not initialize unnecessarily +dotnet_diagnostic.CA1810.severity = suggestion +# CA1822: Make member static +dotnet_diagnostic.CA1822.severity = suggestion +# CA1823: Avoid zero-length array allocations +dotnet_diagnostic.CA1825.severity = suggestion +# CA1826: Do not use Enumerable methods on indexable collections. Instead use the collection directly +dotnet_diagnostic.CA1826.severity = suggestion +# CA1827: Do not use Count() or LongCount() when Any() can be used +dotnet_diagnostic.CA1827.severity = suggestion +# CA1829: Use Length/Count property instead of Count() when available +dotnet_diagnostic.CA1829.severity = suggestion +# CA1831: Use AsSpan or AsMemory instead of Range-based indexers when appropriate +dotnet_diagnostic.CA1831.severity = suggestion +# CA1832: Use AsSpan or AsMemory instead of Range-based indexers when appropriate +dotnet_diagnostic.CA1832.severity = suggestion +# CA1833: Use AsSpan or AsMemory instead of Range-based indexers when appropriate +dotnet_diagnostic.CA1833.severity = suggestion +# CA1834: Consider using 'StringBuilder.Append(char)' when applicable +dotnet_diagnostic.CA1834.severity = suggestion +# CA1835: Prefer the 'Memory'-based overloads for 'ReadAsync' and 'WriteAsync' +dotnet_diagnostic.CA1835.severity = suggestion +# CA1837: Use 'Environment.ProcessId' +dotnet_diagnostic.CA1837.severity = suggestion +# CA1838: Avoid 'StringBuilder' parameters for P/Invokes +dotnet_diagnostic.CA1838.severity = suggestion +# CA1841: Prefer Dictionary.Contains methods +dotnet_diagnostic.CA1841.severity = suggestion +# CA1844: Provide memory-based overrides of async methods when subclassing 'Stream' +dotnet_diagnostic.CA1844.severity = suggestion +# CA1845: Use span-based 'string.Concat' +dotnet_diagnostic.CA1845.severity = suggestion +# CA1846: Prefer AsSpan over Substring +dotnet_diagnostic.CA1846.severity = suggestion +# CA1847: Use string.Contains(char) instead of string.Contains(string) with single characters +dotnet_diagnostic.CA1847.severity = suggestion +# CA1852: Seal internal types +dotnet_diagnostic.CA1852.severity = suggestion +# CA1854: Prefer the IDictionary.TryGetValue(TKey, out TValue) method +dotnet_diagnostic.CA1854.severity = suggestion +# CA1855: Prefer 'Clear' over 'Fill' +dotnet_diagnostic.CA1855.severity = suggestion +# CA1856: Incorrect usage of ConstantExpected attribute +dotnet_diagnostic.CA1856.severity = suggestion +# CA1857: A constant is expected for the parameter +dotnet_diagnostic.CA1857.severity = suggestion +# CA1858: Use 'StartsWith' instead of 'IndexOf' +dotnet_diagnostic.CA1858.severity = suggestion +# CA2007: Consider calling ConfigureAwait on the awaited task +dotnet_diagnostic.CA2007.severity = suggestion +# CA2008: Do not create tasks without passing a TaskScheduler +dotnet_diagnostic.CA2008.severity = suggestion +# CA2012: Use ValueTask correctly +dotnet_diagnostic.CA2012.severity = suggestion +# CA2201: Do not raise reserved exception types +dotnet_diagnostic.CA2201.severity = suggestion +# CA2249: Use string.Contains instead of string.IndexOf to improve readability. +dotnet_diagnostic.CA2249.severity = suggestion +# IDE0005: Remove unnecessary usings +dotnet_diagnostic.IDE0005.severity = suggestion +# IDE0020: Use pattern matching to avoid is check followed by a cast (with variable) +dotnet_diagnostic.IDE0020.severity = suggestion +# IDE0029: Use coalesce expression (non-nullable types) +dotnet_diagnostic.IDE0029.severity = suggestion +# IDE0030: Use coalesce expression (nullable types) +dotnet_diagnostic.IDE0030.severity = suggestion +# IDE0031: Use null propagation +dotnet_diagnostic.IDE0031.severity = suggestion +# IDE0038: Use pattern matching to avoid is check followed by a cast (without variable) +dotnet_diagnostic.IDE0038.severity = suggestion +# IDE0044: Make field readonly +dotnet_diagnostic.IDE0044.severity = suggestion +# IDE0051: Remove unused private members +dotnet_diagnostic.IDE0051.severity = suggestion +# IDE0059: Unnecessary assignment to a value +dotnet_diagnostic.IDE0059.severity = suggestion +# IDE0060: Remove unused parameters +dotnet_diagnostic.IDE0060.severity = suggestion +# IDE0062: Make local function static +dotnet_diagnostic.IDE0062.severity = suggestion +# IDE0200: Lambda expression can be removed +dotnet_diagnostic.IDE0200.severity = suggestion + +# CA2016: Forward the 'CancellationToken' parameter to methods that take one +dotnet_diagnostic.CA2016.severity = suggestion + +# Defaults for content in the shared src/ and shared runtime dir + +[{**/Shared/runtime/**.{cs,vb},src/Shared/test/Shared.Tests/runtime/**.{cs,vb},**/microsoft.extensions.hostfactoryresolver.sources/**.{cs,vb}}] +# CA1822: Make member static +dotnet_diagnostic.CA1822.severity = silent +# IDE0011: Use braces +dotnet_diagnostic.IDE0011.severity = silent +# IDE0055: Fix formatting +dotnet_diagnostic.IDE0055.severity = silent +# IDE0060: Remove unused parameters +dotnet_diagnostic.IDE0060.severity = silent +# IDE0062: Make local function static +dotnet_diagnostic.IDE0062.severity = silent +# IDE0161: Convert to file-scoped namespace +dotnet_diagnostic.IDE0161.severity = silent + +[{**/Shared/**.cs,**/microsoft.extensions.hostfactoryresolver.sources/**.{cs,vb}}] +# IDE0005: Remove unused usings. Ignore for shared src files since imports for those depend on the projects in which they are included. +dotnet_diagnostic.IDE0005.severity = silent \ No newline at end of file diff --git a/Benchmarks/BinaryConversionBenchmarks.cs b/Benchmarks/BinaryConversionBenchmarks.cs index 820502d..f6d7eee 100644 --- a/Benchmarks/BinaryConversionBenchmarks.cs +++ b/Benchmarks/BinaryConversionBenchmarks.cs @@ -14,30 +14,30 @@ using BenchmarkDotNet.Order; [ThreadingDiagnoser] public class BinaryConversionBenchmarks { - private byte[] data; - private int offset; + private byte[] _data; + private int _offset; [GlobalSetup] public void Setup() { - this.data = RandomNumberGenerator.GetBytes(4000); - this.offset = RandomNumberGenerator.GetInt32(0, 3500); + this._data = RandomNumberGenerator.GetBytes(4000); + this._offset = RandomNumberGenerator.GetInt32(0, 3500); } [Benchmark] - public short BitConverterTest() => BitConverter.ToInt16(this.data, this.offset); + public short BitConverterTest() => BitConverter.ToInt16(this._data, this._offset); [Benchmark] public short BinaryReader() { - using var ms = new MemoryStream(this.data); + using var ms = new MemoryStream(this._data); using var reader = new BinaryReader(ms); - reader.BaseStream.Position = this.offset; + reader.BaseStream.Position = this._offset; return reader.ReadInt16(); } [Benchmark] public short BinaryPrimitives() => System.Buffers.Binary.BinaryPrimitives.ReadInt16LittleEndian( - new ArraySegment(this.data, this.offset, sizeof(short))); + new ArraySegment(this._data, this._offset, sizeof(short))); } diff --git a/Continuity.sln b/Continuity.sln index 87391ca..86836c8 100644 --- a/Continuity.sln +++ b/Continuity.sln @@ -4,21 +4,12 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Server", "Server\Server.csp EndProject Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Benchmarks", "Benchmarks\Benchmarks.csproj", "{7D560FA1-A61C-4B67-8300-835CA5814621}" EndProject -Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "build", "build\build.csproj", "{046A3B46-FC08-4B08-A52A-1DC5D426120D}" -EndProject -Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Solution Items", "Solution Items", "{C1303691-99F6-41CE-BAD3-2CA112D75DBF}" - ProjectSection(SolutionItems) = preProject - docker-compose.yml = docker-compose.yml - EndProjectSection -EndProject Global GlobalSection(SolutionConfigurationPlatforms) = preSolution Debug|Any CPU = Debug|Any CPU Release|Any CPU = Release|Any CPU EndGlobalSection GlobalSection(ProjectConfigurationPlatforms) = postSolution - {046A3B46-FC08-4B08-A52A-1DC5D426120D}.Debug|Any CPU.ActiveCfg = Debug|Any CPU - {046A3B46-FC08-4B08-A52A-1DC5D426120D}.Release|Any CPU.ActiveCfg = Release|Any CPU {7EDA8B31-3E03-4CA3-87D1-CFEB05C277D6}.Debug|Any CPU.ActiveCfg = Debug|Any CPU {7EDA8B31-3E03-4CA3-87D1-CFEB05C277D6}.Debug|Any CPU.Build.0 = Debug|Any CPU {7EDA8B31-3E03-4CA3-87D1-CFEB05C277D6}.Release|Any CPU.ActiveCfg = Release|Any CPU diff --git a/Server/AuthSession.cs b/Server/AuthSession.cs index bdc3168..df1dd71 100644 --- a/Server/AuthSession.cs +++ b/Server/AuthSession.cs @@ -8,28 +8,28 @@ using Packets; public class AuthSession : TcpSession { - private readonly ILogger logger; - private readonly IMediator mediator; + private readonly ILogger _logger; + private readonly IMediator _mediator; public AuthSession(TcpServer server, IMediator mediator, ILogger logger) : base(server) { - this.mediator = mediator; - this.logger = logger; + this._mediator = mediator; + this._logger = logger; } public Guid AccountId { get; set; } public override long Send(byte[] buffer) { - this.logger.LogInformation("Data being sent is: {Data}", BitConverter.ToString(buffer)); + this._logger.LogInformation("Data being sent is: {Data}", BitConverter.ToString(buffer)); return base.Send(buffer); } public void Send(IPacket packet) { var type = packet.GetType(); - this.logger.LogTrace("Packet of type {Type} is being serialized", type.Name); + this._logger.LogTrace("Packet of type {Type} is being serialized", type.Name); var packetIdAttribute = type.GetCustomAttribute(); if (packetIdAttribute == null) { @@ -57,40 +57,40 @@ public class AuthSession : TcpSession buffer[2 + i] = bytesOfOpcode[i]; } - this.logger.LogTrace("Packet data being parsed is: {Data}", BitConverter.ToString(packetData.ToArray())); - this.logger.LogTrace("Packet being parsed is: {Data}", BitConverter.ToString(buffer.ToArray())); + this._logger.LogTrace("Packet data being parsed is: {Data}", BitConverter.ToString(packetData.ToArray())); + this._logger.LogTrace("Packet being parsed is: {Data}", BitConverter.ToString(buffer.ToArray())); this.Send(buffer); } protected override void OnReceived(byte[] buffer, long offset, long size) { - this.logger.LogDebug("Length: {Size} & offset: {Offset}", size, offset); + this._logger.LogDebug("Length: {Size} & offset: {Offset}", size, offset); Span decryptedBuffer = new byte[size]; // xor every value after the first 8 bytes var dataBuffer = Decrypt(new ArraySegment(buffer, 8, (int)size - 8).ToArray()); - this.logger.LogDebug("Length {Length}", BitConverter.ToUInt16(buffer, 0)); + this._logger.LogDebug("Length {Length}", BitConverter.ToUInt16(buffer, 0)); var opCode = BitConverter.ToUInt16(buffer.ToArray(), 2); - this.logger.LogDebug("Packet Op Code: {OpCode}", opCode); - this.logger.LogDebug("Some Value: {RandomValue}", buffer[4]); + this._logger.LogDebug("Packet Op Code: {OpCode}", opCode); + this._logger.LogDebug("Some Value: {RandomValue}", buffer[4]); var clientAliveTime = BitConverter.ToUInt16(buffer.ToArray(), 5); - this.logger.LogDebug("Client Alive time: {ClientAliveTime}", clientAliveTime); - this.logger.LogDebug("Might be a flag: {Flag}", buffer[7]); + this._logger.LogDebug("Client Alive time: {ClientAliveTime}", clientAliveTime); + this._logger.LogDebug("Might be a flag: {Flag}", buffer[7]); - this.logger.LogDebug("Full buffer: {Buffer}", BitConverter.ToString(dataBuffer.ToArray())); + this._logger.LogDebug("Full buffer: {Buffer}", BitConverter.ToString(dataBuffer.ToArray())); var rawPacket = new RawPacket((OperationCode)opCode, dataBuffer, clientAliveTime, buffer[7], buffer[4], this.Id, this); - _ = this.mediator.Send(rawPacket); + _ = this._mediator.Send(rawPacket); - this.logger.LogInformation("Connection from: {@RemoteEndpoint}", this.Socket.RemoteEndPoint?.ToString()); + this._logger.LogInformation("Connection from: {@RemoteEndpoint}", this.Socket.RemoteEndPoint?.ToString()); base.OnReceived(decryptedBuffer.ToArray(), offset, decryptedBuffer.Length); } diff --git a/Server/ChannelSession.cs b/Server/ChannelSession.cs index ae11461..593719e 100644 --- a/Server/ChannelSession.cs +++ b/Server/ChannelSession.cs @@ -8,34 +8,34 @@ using NetCoreServer; public class ChannelSession : TcpSession { - private static readonly byte[] Key = new byte[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6, 7 } + private static readonly byte[] _key = new byte[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 1, 2, 3, 4, 5, 6, 7 } .Reverse().ToArray(); - private static readonly byte[] Iv = new byte[] + private static readonly byte[] _iv = new byte[] { 0xfe, 220, 0xba, 0x98, 0x76, 0x54, 50, 0x10, 15, 30, 0x2d, 60, 0x4b, 90, 0x69, 120 }.Reverse().ToArray(); - private readonly ICryptoTransform decryptor; + private readonly ICryptoTransform _decryptor; - private readonly ICryptoTransform encryptor; - private readonly ILogger logger; - private readonly IMediator mediator; + private readonly ICryptoTransform _encryptor; + private readonly ILogger _logger; + private readonly IMediator _mediator; public ChannelSession(TcpServer server, IMediator mediator, ILogger logger) : base(server) { - this.mediator = mediator; - this.logger = logger; + this._mediator = mediator; + this._logger = logger; var aes = Aes.Create(); - aes.Key = Key; - aes.IV = Iv; + aes.Key = _key; + aes.IV = _iv; aes.Padding = PaddingMode.None; #pragma warning disable SEC0026 aes.Mode = CipherMode.ECB; #pragma warning restore SEC0026 - this.decryptor = aes.CreateDecryptor(aes.Key, aes.IV); - this.encryptor = aes.CreateEncryptor(aes.Key, aes.IV); + this._decryptor = aes.CreateDecryptor(aes.Key, aes.IV); + this._encryptor = aes.CreateEncryptor(aes.Key, aes.IV); } protected override void OnReceived(byte[] buffer, long offset, long size) @@ -43,12 +43,12 @@ public class ChannelSession : TcpSession try { using (var ms = new MemoryStream(Decrypt(buffer))) - using (var cs = new CryptoStream(ms, this.decryptor, CryptoStreamMode.Read)) + using (var cs = new CryptoStream(ms, this._decryptor, CryptoStreamMode.Read)) { var amountOfReadBytes = cs.Read(buffer); if (amountOfReadBytes != buffer.Length) { - this.logger.LogError("Amount of read bytes is not equal to buffer length."); + this._logger.LogError("Amount of read bytes is not equal to buffer length."); } } @@ -56,8 +56,8 @@ public class ChannelSession : TcpSession } catch (CryptographicException ex) { - this.logger.LogError("An error has occured while decrypting: {ErrorMessage}", ex.Message); - this.logger.LogError("Default buffer message: {Message}", Encoding.ASCII.GetString(buffer)); + this._logger.LogError("An error has occured while decrypting: {ErrorMessage}", ex.Message); + this._logger.LogError("Default buffer message: {Message}", Encoding.ASCII.GetString(buffer)); } } diff --git a/Server/Consumers/PacketConsumer.cs b/Server/Consumers/PacketConsumer.cs index 44c80cf..9b9f7cd 100644 --- a/Server/Consumers/PacketConsumer.cs +++ b/Server/Consumers/PacketConsumer.cs @@ -6,13 +6,13 @@ using Services; public class PacketConsumer : IConsumer { - private readonly PacketDistributorService distributorService; + private readonly PacketDistributorService _distributorService; - public PacketConsumer(PacketDistributorService distributorService) => this.distributorService = distributorService; + public PacketConsumer(PacketDistributorService distributorService) => this._distributorService = distributorService; public Task Consume(ConsumeContext context) { - this.distributorService.AddPacket(context.Message); + this._distributorService.AddPacket(context.Message); return Task.CompletedTask; } } diff --git a/Server/DB/WonderkingContext.cs b/Server/DB/WonderkingContext.cs index 0b33c01..2cc9720 100644 --- a/Server/DB/WonderkingContext.cs +++ b/Server/DB/WonderkingContext.cs @@ -7,13 +7,13 @@ using Microsoft.Extensions.Logging; public class WonderkingContext : DbContext { - private readonly IConfiguration configuration; - private readonly ILoggerFactory loggerFactory; + private readonly IConfiguration _configuration; + private readonly ILoggerFactory _loggerFactory; public WonderkingContext(ILoggerFactory loggerFactory, IConfiguration configuration) { - this.loggerFactory = loggerFactory; - this.configuration = configuration; + this._loggerFactory = loggerFactory; + this._configuration = configuration; } public DbSet Accounts { get; set; } @@ -22,8 +22,8 @@ public class WonderkingContext : DbContext protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder) => optionsBuilder .UseNpgsql( - $"Host={this.configuration["DB:Host"]};Username={this.configuration["DB:Username"]};Password={this.configuration["DB:Password"]};Database={this.configuration["DB:Database"]};Port={this.configuration["DB:Port"]}") - .EnableSensitiveDataLogging().UseLoggerFactory(this.loggerFactory); + $"Host={this._configuration["DB:Host"]};Username={this._configuration["DB:Username"]};Password={this._configuration["DB:Password"]};Database={this._configuration["DB:Database"]};Port={this._configuration["DB:Port"]}") + .EnableSensitiveDataLogging().UseLoggerFactory(this._loggerFactory); protected override void OnModelCreating(ModelBuilder modelBuilder) => modelBuilder.Entity(builder => diff --git a/Server/LoggerMessages/PacketLoggerMessages.cs b/Server/LoggerMessages/PacketLoggerMessages.cs index 25257b4..0654492 100644 --- a/Server/LoggerMessages/PacketLoggerMessages.cs +++ b/Server/LoggerMessages/PacketLoggerMessages.cs @@ -43,7 +43,6 @@ public static partial class PacketLoggerMessages Message = "Packet data {PacketData}")] public static partial void PacketData(this ILogger logger, string packetData); - [LoggerMessage(EventId = 9, Level = LogLevel.Critical, Message = "No Packets have been found")] public static partial void NoPacketsFound(this ILogger logger); diff --git a/Server/PacketHandlers/ChannelSelectionHandler.cs b/Server/PacketHandlers/ChannelSelectionHandler.cs index a8091b0..3db2358 100644 --- a/Server/PacketHandlers/ChannelSelectionHandler.cs +++ b/Server/PacketHandlers/ChannelSelectionHandler.cs @@ -8,16 +8,16 @@ using Packets.Incoming; public class ChannelSelectionHandler : IPacketHandler { - private readonly IConfiguration configuration; - private readonly ILogger logger; - private readonly WonderkingContext wonderkingContext; + private readonly IConfiguration _configuration; + private readonly ILogger _logger; + private readonly WonderkingContext _wonderkingContext; public ChannelSelectionHandler(IConfiguration configuration, ILogger logger, WonderkingContext wonderkingContext) { - this.configuration = configuration; - this.logger = logger; - this.wonderkingContext = wonderkingContext; + this._configuration = configuration; + this._logger = logger; + this._wonderkingContext = wonderkingContext; } public ChannelSelectionHandler() @@ -27,7 +27,7 @@ public class ChannelSelectionHandler : IPacketHandler public Task HandleAsync(ChannelSelectionPacket packet, TcpSession session) { var authSession = (AuthSession)session; - var charactersOfAccount = this.wonderkingContext.Accounts.FirstOrDefault(a => a.Id == authSession.AccountId); + var charactersOfAccount = this._wonderkingContext.Accounts.FirstOrDefault(a => a.Id == authSession.AccountId); return Task.CompletedTask; } } diff --git a/Server/PacketHandlers/LoginHandler.cs b/Server/PacketHandlers/LoginHandler.cs index dbd1a65..3955229 100644 --- a/Server/PacketHandlers/LoginHandler.cs +++ b/Server/PacketHandlers/LoginHandler.cs @@ -13,23 +13,23 @@ using Packets.Outgoing; public class LoginHandler : IPacketHandler { - private readonly IConfiguration configuration; - private readonly ILogger logger; - private readonly WonderkingContext wonderkingContext; + private readonly IConfiguration _configuration; + private readonly ILogger _logger; + private readonly WonderkingContext _wonderkingContext; public LoginHandler(ILogger logger, WonderkingContext wonderkingContext, IConfiguration configuration) { - this.logger = logger; - this.wonderkingContext = wonderkingContext; - this.configuration = configuration; + this._logger = logger; + this._wonderkingContext = wonderkingContext; + this._configuration = configuration; } public async Task HandleAsync(LoginInfoPacket packet, TcpSession session) { LoginResponseReason loginResponseReason; - this.logger.LogInformation("Login data: Username {Username} & Password {Password}", packet.Username, + this._logger.LogInformation("Login data: Username {Username} & Password {Password}", packet.Username, packet.Password); - var account = this.wonderkingContext.Accounts.FirstOrDefault(a => a.Username == packet.Username); + var account = this._wonderkingContext.Accounts.FirstOrDefault(a => a.Username == packet.Username); // https://cheatsheetseries.owasp.org/cheatsheets/Password_Storage_Chea1t_Sheet.html#argon2id // "Use Argon2id with a minimum configuration of 19 MiB of memory, an iteration count of 2, and 1 degree of parallelism." @@ -41,22 +41,22 @@ public class LoginHandler : IPacketHandler }; if (account == null) { - if (this.configuration.GetSection("Testing").GetValue("CreateAccountOnLogin")) + if (this._configuration.GetSection("Testing").GetValue("CreateAccountOnLogin")) { argon2Id.Salt = RandomNumberGenerator.GetBytes(16); var finalAccount = - await this.wonderkingContext.Accounts.AddAsync(new Account(packet.Username, Array.Empty(), "", + await this._wonderkingContext.Accounts.AddAsync(new Account(packet.Username, Array.Empty(), "", 0, argon2Id.Salt)).ConfigureAwait(true); - await this.wonderkingContext.SaveChangesAsync().ConfigureAwait(true); + await this._wonderkingContext.SaveChangesAsync().ConfigureAwait(true); argon2Id.AssociatedData = finalAccount.Entity.Id.ToByteArray(); finalAccount.Entity.Password = await argon2Id.GetBytesAsync(16).ConfigureAwait(true); - this.wonderkingContext.Accounts.Update(finalAccount.Entity); + this._wonderkingContext.Accounts.Update(finalAccount.Entity); loginResponseReason = LoginResponseReason.Ok; - await this.wonderkingContext.SaveChangesAsync().ConfigureAwait(true); + await this._wonderkingContext.SaveChangesAsync().ConfigureAwait(true); } else { - this.logger.LogInformation("Requested account for user: {Username} does not exist", packet.Username); + this._logger.LogInformation("Requested account for user: {Username} does not exist", packet.Username); loginResponseReason = LoginResponseReason.AccountDoesNotExit; } } diff --git a/Server/Program.cs b/Server/Program.cs index 76327dd..3241823 100644 --- a/Server/Program.cs +++ b/Server/Program.cs @@ -1,4 +1,3 @@ -#pragma warning disable AV1500 using System.Net; using System.Reflection; using MassTransit; @@ -35,11 +34,10 @@ builder.Services.AddHostedService(provider => new WonderkingAuthServer(IPAddress provider.GetService() ?? throw new InvalidOperationException())); using var host = builder.Build(); -await using (var scope = host.Services.CreateAsyncScope()) +using (var scope = host.Services.CreateScope()) { var db = scope.ServiceProvider.GetRequiredService(); await db.Database.MigrateAsync().ConfigureAwait(true); } await host.RunAsync().ConfigureAwait(true); -#pragma warning restore AV1500 diff --git a/Server/Server.csproj b/Server/Server.csproj index 34e1cef..bd69f41 100644 --- a/Server/Server.csproj +++ b/Server/Server.csproj @@ -45,10 +45,6 @@ - - all - runtime; build; native; contentfiles; analyzers; buildtransitive - @@ -61,6 +57,10 @@ runtime; build; native; contentfiles; analyzers; buildtransitive + + all + runtime; build; native; contentfiles; analyzers; buildtransitive + diff --git a/Server/Services/PacketDistributorService.cs b/Server/Services/PacketDistributorService.cs index da2d85c..28cbae4 100644 --- a/Server/Services/PacketDistributorService.cs +++ b/Server/Services/PacketDistributorService.cs @@ -17,35 +17,35 @@ using static DotNext.Metaprogramming.CodeGenerator; public class PacketDistributorService : IHostedService { - private readonly ConcurrentQueue concurrentQueue; + private readonly ConcurrentQueue _concurrentQueue; private readonly ImmutableDictionary> deserializationMap; + Func> _deserializationMap; - private readonly ILogger logger; - private readonly ConcurrentDictionary packetHandlersInstantiation; + private readonly ILogger _logger; + private readonly ConcurrentDictionary _packetHandlersInstantiation; - private readonly IServiceProvider serviceProvider; + private readonly IServiceProvider _serviceProvider; public PacketDistributorService(ILogger logger, IServiceProvider serviceProvider) { - this.concurrentQueue = new ConcurrentQueue(); - this.logger = logger; - this.serviceProvider = serviceProvider; + this._concurrentQueue = new ConcurrentQueue(); + this._logger = logger; + this._serviceProvider = serviceProvider; var tempDeserializationMap = new Dictionary>(); var executingAssembly = Assembly.GetExecutingAssembly(); var packetsTypes = this.GetPacketsWithId(executingAssembly); var packetHandlers = this.GetAllPacketHandlersWithId(executingAssembly); - this.packetHandlersInstantiation = new ConcurrentDictionary(); + this._packetHandlersInstantiation = new ConcurrentDictionary(); packetHandlers.ForEach(x => { var packetHandler = - ActivatorUtilities.GetServiceOrCreateInstance(this.serviceProvider, + ActivatorUtilities.GetServiceOrCreateInstance(this._serviceProvider, x.Value); - this.packetHandlersInstantiation.TryAdd(x.Key, packetHandler); + this._packetHandlersInstantiation.TryAdd(x.Key, packetHandler); }); foreach (var packetsType in packetsTypes) { @@ -64,7 +64,7 @@ public class PacketDistributorService : IHostedService tempDeserializationMap.Add(packetsType.Key, lambda); } - this.deserializationMap = tempDeserializationMap.ToImmutableDictionary(); + this._deserializationMap = tempDeserializationMap.ToImmutableDictionary(); } public Task StartAsync(CancellationToken cancellationToken) => Task.CompletedTask; @@ -82,11 +82,11 @@ public class PacketDistributorService : IHostedService if (packetsWithId is not { Count: 0 }) { packetsWithId.AsParallel() - .ForAll(packet => this.logger.PacketWithIdAdded(packet.Key, packet.Value.FullName)); + .ForAll(packet => this._logger.PacketWithIdAdded(packet.Key, packet.Value.FullName)); return packetsWithId; } - this.logger.NoPacketsFound(); + this._logger.NoPacketsFound(); throw new IncompleteInitialization(); } @@ -102,24 +102,24 @@ public class PacketDistributorService : IHostedService if (packetHandlersWithId is not { Count: 0 }) { packetHandlersWithId.AsParallel().ForAll(packetHandler => - this.logger.PacketHandlerWithIdAdded(packetHandler.Key, packetHandler.Value.FullName)); + this._logger.PacketHandlerWithIdAdded(packetHandler.Key, packetHandler.Value.FullName)); return packetHandlersWithId; } - this.logger.NoPacketHandlersFound(); + this._logger.NoPacketHandlersFound(); throw new IncompleteInitialization(); } public void AddPacket(RawPacket rawPacket) { - this.concurrentQueue.Enqueue(rawPacket); + this._concurrentQueue.Enqueue(rawPacket); this.DequeueRawPacket(); - this.logger.PacketReceived(rawPacket.OperationCode); + this._logger.PacketReceived(rawPacket.OperationCode); } private void DequeueRawPacket() { - if (this.concurrentQueue.TryDequeue(out var item)) + if (this._concurrentQueue.TryDequeue(out var item)) { ThreadPool.QueueUserWorkItem(this.InvokePacketHandler, item, true); } @@ -127,18 +127,18 @@ public class PacketDistributorService : IHostedService private void InvokePacketHandler(RawPacket item) { - this.logger.PacketDequeued(item.Session.Id, item.OperationCode); - if (!this.deserializationMap.TryGetValue(item.OperationCode, out var value)) + this._logger.PacketDequeued(item.Session.Id, item.OperationCode); + if (!this._deserializationMap.TryGetValue(item.OperationCode, out var value)) { - this.logger.PacketTypeNotFound(item.OperationCode); + this._logger.PacketTypeNotFound(item.OperationCode); return; } var packet = value(item.MessageBody); - this.logger.PacketData(JsonConvert.SerializeObject(packet)); - this.packetHandlersInstantiation[item.OperationCode].GetType().GetMethod(nameof(IPacketHandler.HandleAsync)) - ?.Invoke(this.packetHandlersInstantiation[item.OperationCode], new object[] { packet, item.Session }); + this._logger.PacketData(JsonConvert.SerializeObject(packet)); + this._packetHandlersInstantiation[item.OperationCode].GetType().GetMethod(nameof(IPacketHandler.HandleAsync)) + ?.Invoke(this._packetHandlersInstantiation[item.OperationCode], new object[] { packet, item.Session }); - this.logger.PacketFinished(item.Session.Id, item.OperationCode); + this._logger.PacketFinished(item.Session.Id, item.OperationCode); } } diff --git a/Server/Services/WonderkingAuthServer.cs b/Server/Services/WonderkingAuthServer.cs index 6cbb8e7..fd9d6e1 100644 --- a/Server/Services/WonderkingAuthServer.cs +++ b/Server/Services/WonderkingAuthServer.cs @@ -9,14 +9,14 @@ using NetCoreServer; public class WonderkingAuthServer : TcpServer, IHostedService { - private readonly ILogger logger; - private readonly IServiceProvider serviceProvider; + private readonly ILogger _logger; + private readonly IServiceProvider _serviceProvider; public WonderkingAuthServer(IPAddress address, int port, ILogger logger, IServiceProvider serviceProvider) : base(address, port) { - this.logger = logger; - this.serviceProvider = serviceProvider; + this._logger = logger; + this._serviceProvider = serviceProvider; } public Task StartAsync(CancellationToken cancellationToken) @@ -32,31 +32,31 @@ public class WonderkingAuthServer : TcpServer, IHostedService } protected override TcpSession CreateSession() => - ActivatorUtilities.CreateInstance(this.serviceProvider, this); + ActivatorUtilities.CreateInstance(this._serviceProvider, this); protected override void OnStarting() { - this.logger.LogInformation("Starting"); + this._logger.LogInformation("Starting"); base.OnStarting(); } protected override void OnStarted() { - this.logger.LogInformation("Started"); + this._logger.LogInformation("Started"); base.OnStarted(); } protected override void OnStopping() { - this.logger.LogInformation("Stopping"); + this._logger.LogInformation("Stopping"); base.OnStopping(); } protected override void OnStopped() { - this.logger.LogInformation("Stopped"); + this._logger.LogInformation("Stopped"); base.OnStopped(); } - protected override void OnError(SocketError error) => this.logger.LogError("An error has occured {Error}", error); + protected override void OnError(SocketError error) => this._logger.LogError("An error has occured {Error}", error); } diff --git a/build/.editorconfig b/build/.editorconfig deleted file mode 100644 index 31e43dc..0000000 --- a/build/.editorconfig +++ /dev/null @@ -1,11 +0,0 @@ -[*.cs] -dotnet_style_qualification_for_field = false:warning -dotnet_style_qualification_for_property = false:warning -dotnet_style_qualification_for_method = false:warning -dotnet_style_qualification_for_event = false:warning -dotnet_style_require_accessibility_modifiers = never:warning - -csharp_style_expression_bodied_methods = true:silent -csharp_style_expression_bodied_properties = true:warning -csharp_style_expression_bodied_indexers = true:warning -csharp_style_expression_bodied_accessors = true:warning diff --git a/build/Build.cs b/build/Build.cs deleted file mode 100644 index 19e8d68..0000000 --- a/build/Build.cs +++ /dev/null @@ -1,116 +0,0 @@ -using Nuke.Common; -using Nuke.Common.Git; -using Nuke.Common.IO; -using Nuke.Common.Tools.Docker; -using Nuke.Common.Tools.DotNet; -using Nuke.Common.Tools.SonarScanner; -using Serilog; - -sealed class Build : NukeBuild -{ - /// Support plugins are available for: - /// - JetBrains ReSharper https://nuke.build/resharper - /// - JetBrains Rider https://nuke.build/rider - /// - Microsoft VisualStudio https://nuke.build/visualstudio - /// - Microsoft VSCode https://nuke.build/vscode - public static int Main() => Execute(x => x.Information, x => x.Deploy); - - [Parameter("Configuration to build - Default is 'Debug' (local) or 'Release' (server)")] - readonly Configuration Configuration = IsLocalBuild ? Configuration.Debug : Configuration.Release; - - [Parameter] readonly bool SupportsDocker = true; - - [Parameter] readonly string SonarToken; - [Parameter] readonly string SonarHostUrl; - - GitRepository GitRepository => GitRepository.FromLocalDirectory(RootDirectory); - - Target Information => _ => _ - .Before(Clean) - .Executes(() => - { - Log.Information("Branch Name: {BranchName}", GitRepository.Branch); - Log.Information("Commit: {Commit}", GitRepository.Commit); - }); - - Target Clean => _ => _ - .Before(Restore) - .Executes(() => DotNetTasks.DotNetClean()); - - Target Restore => _ => _ - .Executes(() => DotNetTasks.DotNetRestore()); - - Target Compile => _ => _ - .DependsOn(Clean) - .DependsOn(Restore) - .Executes(() => - { - if (IsLocalBuild && SupportsDocker) - { - DockerTasks.DockerStackRm(settings => settings.SetStacks("continuity")); - } - - DotNetTasks.DotNetBuild(settings => settings.SetNoRestore(true)); - }); - - Target Pack => _ => _ - .DependsOn(SonarScan) - .Executes(() => - { - if (!SupportsDocker) - { - return; - } - - DockerTasks.DockerBuild(settings => settings - .SetPath(AbsolutePath.Create(RootDirectory)) - .SetFile("Server/Dockerfile") - .SetTag("continuity-server:latest") - .SetCompress(true) - .SetProgress(ProgressType.auto)); - if (!IsLocalBuild) - { - //DockerTasks.DockerPush(settings => settings.SetName("continuity-server:latest")); - } - }); - - Target SonarScan => _ => _.DependsOn(Compile).Executes(() => - { - if (IsLocalBuild) - { - return; - } - - - var sonarScannerSettings = new SonarScannerBeginSettings() - .SetProjectKey("wonderking_continuity_AYeecUUTs-PH__JrTRky") - .SetLogin(SonarToken) - .SetServer(SonarHostUrl); - var sonarScannerEndSettings = new SonarScannerEndSettings().SetLogin(SonarToken); - SonarScannerTasks.SonarScannerBegin(sonarScannerSettings); - DotNetTasks.DotNetBuild(settings => settings.SetNoRestore(true)); - SonarScannerTasks.SonarScannerEnd(sonarScannerEndSettings); - }); - - Target Deploy => _ => _.DependsOn(Pack).Executes(() => - { - if (!SupportsDocker) - { - return; - } - - DockerTasks.DockerPull(settings => settings.SetName("postgres:16.0-alpine")); - if (IsLocalBuild) - { - var dockerComposeFile = AbsolutePath.Create(RootDirectory / "Server" / "docker-compose.yml"); - DockerTasks.DockerStackDeploy(settings => settings - .AddComposeFile(dockerComposeFile) - .SetStack("continuity") - .SetPrune(true) - ); - return; - } - - Log.Error("Deploy is only available on local builds."); - }); -} diff --git a/build/Configuration.cs b/build/Configuration.cs deleted file mode 100644 index 5519911..0000000 --- a/build/Configuration.cs +++ /dev/null @@ -1,11 +0,0 @@ -using System.ComponentModel; -using Nuke.Common.Tooling; - -[TypeConverter(typeof(TypeConverter))] -public class Configuration : Enumeration -{ - public static Configuration Debug { get; } = new() { Value = nameof(Debug) }; - public static Configuration Release { get; } = new() { Value = nameof(Release) }; - - public static implicit operator string(Configuration configuration) => configuration.Value; -} diff --git a/build/Directory.Build.props b/build/Directory.Build.props deleted file mode 100644 index e147d63..0000000 --- a/build/Directory.Build.props +++ /dev/null @@ -1,8 +0,0 @@ - - - - - - - - diff --git a/build/Directory.Build.targets b/build/Directory.Build.targets deleted file mode 100644 index 2532609..0000000 --- a/build/Directory.Build.targets +++ /dev/null @@ -1,8 +0,0 @@ - - - - - - - - diff --git a/build/build.csproj b/build/build.csproj deleted file mode 100644 index 4d2fb7f..0000000 --- a/build/build.csproj +++ /dev/null @@ -1,23 +0,0 @@ - - - - Exe - net7.0 - - CS0649;CS0169;CA1050;CA1822;CA2211;IDE1006 - .. - .. - 1 - false - - - - - - - - - - - - diff --git a/build/build.csproj.DotSettings b/build/build.csproj.DotSettings deleted file mode 100644 index eb3f4c2..0000000 --- a/build/build.csproj.DotSettings +++ /dev/null @@ -1,28 +0,0 @@ - - DO_NOT_SHOW - DO_NOT_SHOW - DO_NOT_SHOW - DO_NOT_SHOW - DO_NOT_SHOW - Implicit - Implicit - ExpressionBody - 0 - NEXT_LINE - True - False - 120 - IF_OWNER_IS_SINGLE_LINE - WRAP_IF_LONG - False - <Policy Inspect="True" Prefix="" Suffix="" Style="AaBb" /> - <Policy Inspect="True" Prefix="" Suffix="" Style="AaBb" /> - True - True - True - True - True - True - True - True - True