fix(request-info): restore stream position and update to .NET 10

Restores the request body stream position after accessing form data in AspNetCore and Web platforms. This commit also updates the test project to target .NET 10 and migrates the copyright notice to use a dynamic year.

Author: niemyjski

build/common.props

@@ -8,7 +8,7 @@
     <MinVerTagPrefix>v</MinVerTagPrefix>
     <EnableWindowsTargeting>true</EnableWindowsTargeting>
 
-    <Copyright>Copyright (c) 2025 Exceptionless.  All rights reserved.</Copyright>
+    <Copyright>Copyright © $([System.DateTime]::Now.ToString(yyyy)) Exceptionless.  All rights reserved.</Copyright>
     <Authors>Exceptionless</Authors>
     <NoWarn>$(NoWarn);CS1591;NU1701</NoWarn>
     <WarningsAsErrors>true</WarningsAsErrors>

src/Platforms/Exceptionless.AspNetCore/RequestInfoCollector.cs

@@ -62,7 +62,7 @@ private static object GetPostData(HttpContext context, ExceptionlessConfiguratio
             var log = config.Resolver.GetLog();
 
             long contentLength = context.Request.ContentLength.GetValueOrDefault();
-            if(contentLength == 0) {
+            if (contentLength == 0) {
                 string message = "Content-length was zero, empty post.";
                 log.Debug(message);
                 return message;
@@ -95,8 +95,11 @@ private static object GetPostData(HttpContext context, ExceptionlessConfiguratio
                     return message;
                 }
 
+                // Form check must come after seekability and position checks above: accessing
+                // Request.Form triggers reading the request body stream.
                 if (context.Request.HasFormContentType && context.Request.Form.Count > 0) {
                     log.Debug("Reading POST data from Request.Form");
+                    context.Request.Body.Position = originalPosition;
                     return context.Request.Form.ToDictionary(exclusionList);
                 }
 

src/Platforms/Exceptionless.Web/RequestInfoCollector.cs

@@ -108,9 +108,11 @@ private static object GetPostData(HttpContextBase context, ExceptionlessConfigur
                     return message;
                 }
 
+                // Form check must come after seekability and position checks above: accessing
+                // Request.Form triggers reading the request body stream.
                 if (context.Request.Form.Count > 0) {
                     log.Debug("Reading POST data from Request.Form");
-
+                    context.Request.InputStream.Position = originalPosition;
                     return context.Request.Form.ToDictionary(exclusionList);
                 }
 

src/Platforms/Exceptionless.WebApi/RequestInfoCollector.cs

@@ -51,7 +51,7 @@ public static RequestInfo Collect(HttpActionContext context, ExceptionlessConfig
             // POST data collection is not implemented for WebApi due to async complexities and
             // the difficulty of reading the request body without interfering with model binding.
             // Other platforms (AspNetCore, Web) now only collect POST data for unhandled errors.
-            // TODO: support getting post data asyncly.
+            // TODO: support getting post data asynchronously.
             //if (config.IncludePostData && isUnhandledError && context.Request.Method != HttpMethod.Get)
             //    info.PostData = GetPostData(context, config, exclusionList);
 

test/Exceptionless.Tests/Exceptionless.Tests.csproj

@@ -2,10 +2,10 @@
   <Import Project="..\..\build\common.props" />
 
   <PropertyGroup Condition=" '$(OS)' != 'Windows_NT' ">
-<TargetFramework>net8.0</TargetFramework>
+<TargetFramework>net10.0</TargetFramework>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(OS)' == 'Windows_NT' ">
-<TargetFrameworks>net8.0;net462</TargetFrameworks>
+<TargetFrameworks>net10.0;net462</TargetFrameworks>
   </PropertyGroup>
 
   <PropertyGroup>
@@ -19,7 +19,7 @@
     <None Include="app.config" />
   </ItemGroup>
 
-<PropertyGroup Condition=" '$(TargetFramework)' == 'net8.0' " Label="Build">
+<PropertyGroup Condition=" '$(TargetFramework)' == 'net10.0' " Label="Build">
     <DefineConstants>$(DefineConstants);NETSTANDARD;NETSTANDARD2_0</DefineConstants>
   </PropertyGroup>
 

test/Exceptionless.Tests/Platforms/AspNetCoreRequestInfoTests.cs

@@ -1,39 +1,62 @@
+using System.Collections.Generic;
 using System.IO;
 using System.Text;
 using Exceptionless;
 using Exceptionless.Dependency;
 using Microsoft.AspNetCore.Http;
+using Microsoft.Extensions.Primitives;
 using Xunit;
 
 namespace Exceptionless.Tests.Platforms {
     public class AspNetCoreRequestInfoTests {
         [Fact]
         public void GetRequestInfo_DoesNotReadPostData_ForHandledErrors() {
+            // Arrange
             var context = CreateHttpContext("hello=world");
             var config = new ExceptionlessConfiguration(DependencyResolver.CreateDefault());
 
+            // Act
             var requestInfo = context.GetRequestInfo(config);
 
+            // Assert
             Assert.NotNull(requestInfo);
             Assert.Null(requestInfo.PostData);
             Assert.Equal(0L, context.Request.Body.Position);
         }
 
         [Fact]
         public void GetRequestInfo_ReadsAndRestoresPostData_ForUnhandledErrors() {
+            // Arrange
             const string body = "{\"hello\":\"world\"}";
             var context = CreateHttpContext(body);
             var config = new ExceptionlessConfiguration(DependencyResolver.CreateDefault());
 
             context.Request.Body.Position = 5;
 
+            // Act
             var requestInfo = context.GetRequestInfo(config, isUnhandledError: true);
 
+            // Assert
             Assert.NotNull(requestInfo);
             Assert.Equal(body, Assert.IsType<string>(requestInfo.PostData));
             Assert.Equal(5L, context.Request.Body.Position);
         }
 
+        [Fact]
+        public void GetRequestInfo_ReadsFormData_ForUnhandledErrors() {
+            // Arrange
+            var context = CreateFormHttpContext();
+            var config = new ExceptionlessConfiguration(DependencyResolver.CreateDefault());
+
+            // Act
+            var requestInfo = context.GetRequestInfo(config, isUnhandledError: true);
+
+            // Assert
+            Assert.NotNull(requestInfo);
+            var postData = Assert.IsType<Dictionary<string, string>>(requestInfo.PostData);
+            Assert.Equal("world", postData["name"]);
+        }
+
         private static DefaultHttpContext CreateHttpContext(string body) {
             var bodyBytes = Encoding.UTF8.GetBytes(body);
             var context = new DefaultHttpContext();
@@ -43,5 +66,15 @@ private static DefaultHttpContext CreateHttpContext(string body) {
             context.Request.ContentLength = bodyBytes.Length;
             return context;
         }
+
+        private static DefaultHttpContext CreateFormHttpContext() {
+            var context = new DefaultHttpContext();
+            context.Request.Method = HttpMethods.Post;
+            context.Request.ContentType = "application/x-www-form-urlencoded";
+            context.Request.Form = new FormCollection(new Dictionary<string, StringValues> {
+                ["name"] = "world"
+            });
+            return context;
+        }
     }
 }