0.16.2, optimizations, fixes and improvements

src/Entity/MultipartFormCollection.cs

@@ -8,12 +8,7 @@
 // Repository:  https://github.com/sisk-http/core
 
 using Sisk.Core.Internal;
-using System;
 using System.Collections;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
 
 namespace Sisk.Core.Entity;
 

src/Entity/MultipartObject.cs

@@ -8,7 +8,6 @@
 // Repository:  https://github.com/sisk-http/core
 
 using Sisk.Core.Http;
-using Sisk.Core.Internal;
 using System.Collections.Specialized;
 using System.Text;
 

src/Entity/StringValue.cs

@@ -4,16 +4,9 @@
 // The code below is licensed under the MIT license as
 // of the date of its publication, available at
 //
-// File name:   ValueItem.cs
+// File name:   StringValue.cs
 // Repository:  https://github.com/sisk-http/core
 
-using Sisk.Core.Internal;
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-
 namespace Sisk.Core.Entity;
 
 /// <summary>
@@ -272,9 +265,24 @@ void ThrowIfNull()
         }
     }
 
+    /// <inheritdoc/>
     /// <nodocs/>
     public static implicit operator string?(StringValue i)
     {
         return i.Value;
     }
+
+    /// <inheritdoc/>
+    /// <nodocs/>
+    public static bool operator ==(StringValue i, string? other)
+    {
+        return i.Value == other;
+    }
+
+    /// <inheritdoc/>
+    /// <nodocs/>
+    public static bool operator !=(StringValue i, string? other)
+    {
+        return i.Value != other;
+    }
 }

src/Entity/StringValueCollection.cs

@@ -4,17 +4,11 @@
 // The code below is licensed under the MIT license as
 // of the date of its publication, available at
 //
-// File name:   KeyValueCollection.cs
+// File name:   StringValueCollection.cs
 // Repository:  https://github.com/sisk-http/core
 
-using System;
 using System.Collections;
-using System.Collections.Generic;
 using System.Collections.Specialized;
-using System.Diagnostics.CodeAnalysis;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
 
 namespace Sisk.Core.Entity;
 

src/Http/CookieHelpers.cs

@@ -7,6 +7,7 @@
 // File name:   CookieHelpers.cs
 // Repository:  https://github.com/sisk-http/core
 
+using Sisk.Core.Internal;
 using System.Web;
 
 namespace Sisk.Core.Http;
@@ -101,7 +102,7 @@ public void SetCookie(string name, string value, DateTime? expires, TimeSpan? ma
             syntax.Add($"SameSite={sameSite}");
         }
 
-        SetCookieHeader("Set-Cookie", String.Join("; ", syntax));
+        SetCookieHeader(HttpKnownHeaderNames.SetCookie, String.Join("; ", syntax));
     }
 
 }

src/Http/Handlers/DefaultHttpServerHandler.cs

@@ -8,11 +8,6 @@
 // Repository:  https://github.com/sisk-http/core
 
 using Sisk.Core.Routing;
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
 
 namespace Sisk.Core.Http.Handlers;
 

src/Http/Handlers/HttpServerHandler.cs

@@ -12,15 +12,15 @@
 namespace Sisk.Core.Http.Handlers;
 
 /// <summary>
-/// Represents a handler for the <see cref="HttpServer"/>, router, and related modules.
+/// Represents an event handler for the <see cref="HttpServer"/>, router, and related events.
 /// </summary>
 /// <definition>
-/// public class HttpServerHandler
+/// public abstract class HttpServerHandler
 /// </definition>
 /// <type>
 /// Class
 /// </type>
-public class HttpServerHandler
+public abstract class HttpServerHandler
 {
     /// <summary>
     /// Method that is called immediately before starting the <see cref="HttpServer"/>.
@@ -99,7 +99,8 @@ public virtual void OnHttpRequestOpen(HttpRequest request) { }
     public virtual void OnHttpRequestClose(HttpServerExecutionResult result) { }
 
     /// <summary>
-    /// Method that is called when an exception is caught in the Http server.
+    /// Method that is called when an exception is caught in the Http server. This method is called
+    /// regardless of whether <see cref="HttpServerConfiguration.ThrowExceptions"/> is enabled or not.
     /// </summary>
     /// <param name="exception">The exception object.</param>
     /// <definition>

src/Http/Handlers/HttpServerHandlerRepository.cs

@@ -17,12 +17,12 @@ internal class HttpServerHandlerRepository
 
     public void RegisterHandler(HttpServerHandler handler)
     {
-        this.handlers.Add(handler);
+        handlers.Add(handler);
     }
 
     private void CallEvery(Action<HttpServerHandler> action)
     {
-        foreach (HttpServerHandler handler in this.handlers)
+        foreach (HttpServerHandler handler in handlers)
             action(handler);
     }
 

src/Http/Hosting/HttpServerHostContext.cs

@@ -8,7 +8,6 @@
 // Repository:  https://github.com/sisk-http/core
 
 using Sisk.Core.Entity;
-using Sisk.Core.Internal;
 using Sisk.Core.Routing;
 
 namespace Sisk.Core.Http.Hosting;
@@ -66,8 +65,9 @@ public class HttpServerHostContext
     /// <type>
     /// Property
     /// </type>
-    public CrossOriginResourceSharingHeaders CrossOriginResourceSharingPolicy { 
-        get => HttpServer.ServerConfiguration.ListeningHosts[0].CrossOriginResourceSharingPolicy; 
+    public CrossOriginResourceSharingHeaders CrossOriginResourceSharingPolicy
+    {
+        get => HttpServer.ServerConfiguration.ListeningHosts[0].CrossOriginResourceSharingPolicy;
         set => HttpServer.ServerConfiguration.ListeningHosts[0].CrossOriginResourceSharingPolicy = value;
     }
 
@@ -133,7 +133,7 @@ public void Start(bool verbose = true, bool preventHault = true)
             Console.WriteLine(SR.Httpserver_StartMessage);
             foreach (string prefix in HttpServer.ListeningPrefixes)
                 Console.WriteLine("- {0}", prefix);
-        }            
+        }
 
         if (preventHault)
             Thread.Sleep(-1);

src/Http/Hosting/HttpServerHostContextBuilder.cs

@@ -9,7 +9,6 @@
 
 using Sisk.Core.Entity;
 using Sisk.Core.Http.Handlers;
-using Sisk.Core.Internal;
 using Sisk.Core.Routing;
 using System.Diagnostics.CodeAnalysis;
 using System.Globalization;
@@ -140,12 +139,12 @@ public void UsePortableConfiguration(Action<PortableConfigurationBuilder> portab
     /// </summary>
     /// <param name="port">The port the server will listen on.</param>
     /// <definition>
-    /// public void UseListeningPort(int port)
+    /// public void UseListeningPort(ushort port)
     /// </definition>
     /// <type>
     /// Method
     /// </type>
-    public void UseListeningPort(int port)
+    public void UseListeningPort(ushort port)
     {
         _context.ServerConfiguration.ListeningHosts[0].Ports[0] = new ListeningPort(port);
     }

src/Http/Hosting/InitializationParameterCollection.cs

@@ -87,7 +87,8 @@ public class InitializationParameterCollection : IDictionary<string, string?>
                     mappingValue = Parseable.ParseInternal(value, propType)!;
                 }
 #endif
-            } else
+            }
+            else
             {
                 throw new InvalidCastException(string.Format(SR.InitializationParameterCollection_MapCastException, value, propType.FullName));
             }

src/Http/Hosting/PortableConfigurationRequireSection.cs

@@ -7,12 +7,6 @@
 // File name:   PortableConfigurationRequireSection.cs
 // Repository:  https://github.com/sisk-http/core
 
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-
 namespace Sisk.Core.Http.Hosting;
 
 /// <summary>

src/Http/HttpContextBagRepository.cs

@@ -7,7 +7,6 @@
 // File name:   HttpContextBagRepository.cs
 // Repository:  https://github.com/sisk-http/core
 
-using Sisk.Core.Internal;
 using System.Collections;
 using System.Diagnostics.CodeAnalysis;
 
@@ -27,7 +26,7 @@ public class HttpContextBagRepository : IDictionary<string, object?>
     private readonly Dictionary<string, object?> _values = new();
 
     private static string GetTypeKeyName(Type t) =>
-        t.Name + "+" + t.GetHashCode();
+        t.FullName + "+" + t.GetHashCode();
 
     /// <summary>
     /// Creates an new instance of the <see cref="HttpContextBagRepository"/> class.
@@ -254,6 +253,26 @@ public Boolean TryGetValue(String key, [MaybeNullWhen(false)] out Object? value)
         return b;
     }
 
+    /// <summary>
+    /// Gets the value associated with the specified key and casts it into <typeparamref name="TResult"/>.
+    /// </summary>
+    /// <typeparam name="TResult">The type which will be casted into.</typeparam>
+    /// <param name="key">The key whose to get.</param>
+    /// <param name="value">When this method returns, the value associated with the specified key, if the key is found; otherwise, the default value for the type of the value parameter. This parameter is passed uninitialized.</param>
+    /// <returns>true if the object is find with the specified key; otherwise, false.</returns>
+    /// <definition>
+    /// public Boolean TryGetValue{{TResult}}(String key, [MaybeNullWhen(false)] out TResult? value)
+    /// </definition>
+    /// <type>
+    /// Method
+    /// </type>
+    public Boolean TryGetValue<TResult>(String key, [MaybeNullWhen(false)] out TResult? value)
+    {
+        bool b = _values.TryGetValue(key, out Object? v);
+        value = (TResult?)v;
+        return b;
+    }
+
     /// <inheritdoc />
     /// <nodoc />
     IEnumerator IEnumerable.GetEnumerator()

src/Http/HttpRequest.cs

@@ -190,7 +190,7 @@ public NameValueCollection Cookies
                 if (cookies == null)
                 {
                     cookies = new NameValueCollection();
-                    string? cookieHeader = listenerRequest.Headers["cookie"];
+                    string? cookieHeader = listenerRequest.Headers[HttpKnownHeaderNames.Cookie];
                     if (cookieHeader != null)
                     {
                         string[] cookieParts = cookieHeader.Split(';');
@@ -429,15 +429,15 @@ public StringValueCollection Query
         /// public IPAddress Origin { get; }
         /// </definition>
         /// <type>
-        /// Property
+        /// Property 
         /// </type>
         public IPAddress RemoteAddress
         {
             get
             {
                 if (contextServerConfiguration.ResolveForwardedOriginAddress)
                 {
-                    string? forwardedIp = listenerRequest.Headers["X-Forwarded-For"];
+                    string? forwardedIp = listenerRequest.Headers[HttpKnownHeaderNames.XForwardedFor];
                     if (forwardedIp != null)
                     {
                         string forwardedIpLiteralStr = forwardedIp.Contains(',') ? forwardedIp.Substring(forwardedIp.IndexOf(',') + 1) : forwardedIp;
@@ -702,16 +702,6 @@ public HttpResponse Close()
             return new HttpResponse(HttpResponse.HTTPRESPONSE_SERVER_REFUSE);
         }
 
-        /// <summary>
-        /// 
-        /// </summary>
-        /// <param name="statusCode"></param>
-        public void Send(HttpStatusCode statusCode)
-        {
-            byte[] data = Encoding.UTF8.GetBytes($"HTTP/1.1 {(int)statusCode} {statusCode}\n\n");
-            listenerResponse.OutputStream.Write(data);
-        }
-
         /// <summary>
         /// Gets the HTTP request content stream. This property is only available while the
         /// content has not been imported by the HTTP server and will invalidate the body content 

src/Http/HttpResponse.cs

@@ -7,7 +7,6 @@
 // File name:   HttpResponse.cs
 // Repository:  https://github.com/sisk-http/core
 
-using Sisk.Core.Internal;
 using Sisk.Core.Routing;
 using System.Collections.Specialized;
 using System.Net;
@@ -271,6 +270,24 @@ public HttpResponse WithStatus(int status)
             return this;
         }
 
+        /// <summary>
+        /// Sets the HTTP response status information from an <see cref="HttpStatusInformation"/> object.
+        /// </summary>
+        /// <param name="statusInformation">The status information.</param>
+        /// <returns>The self <see cref="HttpResponse"/> object.</returns>
+        /// <definition>
+        /// public HttpResponse WithStatus(int status)
+        /// </definition>
+        /// <type>
+        /// Method
+        /// </type>
+        public HttpResponse WithStatus(HttpStatusInformation statusInformation)
+        {
+            this.Status = default;
+            this.CustomStatus = statusInformation;
+            return this;
+        }
+
         /// <summary>
         /// Sets the HTTP response status code.
         /// </summary>