Add support for Content-Length header parameter support using property source BODY.$LENGTH

C0nquistadore · C0nquistadore · commit 418237ce4e53 · 2025-10-15T13:45:01.000+02:00
diff --git a/shared/Http/SpecialHttpParameterName.cs b/shared/Http/SpecialHttpParameterName.cs
@@ -5,5 +5,6 @@ internal static class SpecialHttpParameterName
         public const string Body = "body";
         public const string MediaType = "mediaType";
         public const string FileName = "fileName";
+        public const string Length = "length";
     }
 }
diff --git a/shared/ParameterSources/BodyParameterSource.cs b/shared/ParameterSources/BodyParameterSource.cs
@@ -6,5 +6,6 @@ internal sealed class BodyParameterSource : ActionParameterSourceDefinition<Body
         public const string RawPropertyName = "$RAW";
         public const string MediaTypePropertyName = "$MEDIATYPE";
         public const string FileNamePropertyName = "$FILENAME";
+        public const string LengthPropertyName = "$LENGTH";
     }
 }
diff --git a/src/Dibix.Http.Host/Model/HttpRequestDescriptor.cs b/src/Dibix.Http.Host/Model/HttpRequestDescriptor.cs
@@ -28,6 +28,8 @@ public HttpRequestDescriptor(HttpRequest request)
 
         public string? GetBodyFileName() => _request.GetTypedHeaders().ContentDisposition?.FileName.Value;
 
+        public long? GetBodyLength() => _request.ContentLength;
+
         public IEnumerable<string> GetHeaderValues(string name) => _request.Headers[name];
 
         public IEnumerable<string> GetAcceptLanguageValues() => _request.GetTypedHeaders().AcceptLanguage.Select(x => x.Value.Value).Where(x => x != null).Select(x => x!);
diff --git a/src/Dibix.Http.Server.AspNet/HttpRequestMessageDescriptor.cs b/src/Dibix.Http.Server.AspNet/HttpRequestMessageDescriptor.cs
@@ -24,6 +24,8 @@ public HttpRequestMessageDescriptor(HttpRequestMessage request)
 
         public string GetBodyFileName() => RequestMessage.Content?.Headers.ContentDisposition?.FileName;
 
+        public long? GetBodyLength() => RequestMessage.Content?.Headers.ContentLength;
+
         public IEnumerable<string> GetHeaderValues(string name) => RequestMessage.Headers.TryGetValues(name, out IEnumerable<string> values) ? values : [];
 
         public IEnumerable<string> GetAcceptLanguageValues() => RequestMessage.Headers.AcceptLanguage.Select(x => x.Value);
diff --git a/src/Dibix.Http.Server/Model/IHttpRequestDescriptor.cs b/src/Dibix.Http.Server/Model/IHttpRequestDescriptor.cs
@@ -11,6 +11,7 @@ public interface IHttpRequestDescriptor
         Stream GetBody();
         string GetBodyMediaType();
         string GetBodyFileName();
+        long? GetBodyLength();
         IEnumerable<string> GetHeaderValues(string name);
         IEnumerable<string> GetAcceptLanguageValues();
         ClaimsPrincipal GetUser();
diff --git a/src/Dibix.Http.Server/Providers/BodyParameterSourceProvider.cs b/src/Dibix.Http.Server/Providers/BodyParameterSourceProvider.cs
@@ -32,6 +32,13 @@ public override void Resolve(IHttpParameterResolutionContext context)
                     break;
                 }
 
+                case BodyParameterSource.LengthPropertyName:
+                {
+                    Expression getBodyLengthCall = Expression.Call(context.RequestParameter, nameof(IHttpRequestDescriptor.GetBodyLength), Type.EmptyTypes);
+                    context.ResolveUsingValue(getBodyLengthCall);
+                    break;
+                }
+
                 default:
                 {
                     Type instanceType = context.ActionMetadata.SafeGetBodyContract();
diff --git a/src/Dibix.Sdk.CodeGeneration/Lookup/ActionTargetDefinitionResolver.cs b/src/Dibix.Sdk.CodeGeneration/Lookup/ActionTargetDefinitionResolver.cs
@@ -101,7 +101,7 @@ protected static bool IsUserParameter(ActionParameterSourceDefinition source, st
                     location = ActionParameterLocation.Path;
                     return true;
 
-                case BodyParameterSource _:
+                case BodyParameterSource _ when propertyName != BodyParameterSource.LengthPropertyName:
                     location = ActionParameterLocation.Body;
                     return true;
 
@@ -169,6 +169,7 @@ private ActionParameter CreateActionParameter(string name, TypeReference type, b
                             {
                                 BodyParameterSource.MediaTypePropertyName => SpecialHttpParameterName.MediaType,
                                 BodyParameterSource.FileNamePropertyName => SpecialHttpParameterName.FileName,
+                                BodyParameterSource.LengthPropertyName => SpecialHttpParameterName.Length,
                                 _ => apiParameterName
                             };
                             break;
diff --git a/src/Dibix.Sdk.CodeGeneration/Output/ApiClientWriter.cs b/src/Dibix.Sdk.CodeGeneration/Output/ApiClientWriter.cs
@@ -72,7 +72,8 @@ protected static void AddMethod(ActionDefinition action, CodeGenerationContext c
                     // No request body contract members
                     case ActionParameterLocation.Body when parameter.ApiParameterName is not (SpecialHttpParameterName.Body
                                                                                            or SpecialHttpParameterName.MediaType
-                                                                                           or SpecialHttpParameterName.FileName):
+                                                                                           or SpecialHttpParameterName.FileName
+                                                                                           or SpecialHttpParameterName.Length):
 
                     // Will be handled by SecurityScheme/IHttpAuthorizationProvider
                     case ActionParameterLocation.Header when parameter.ApiParameterName == "Authorization" || action.SecuritySchemes.Requirements.Any(x => x.Scheme.SchemeName == parameter.ApiParameterName):
diff --git a/src/Dibix.Sdk.CodeGeneration/Registration/ControllerDefinitionProvider.cs b/src/Dibix.Sdk.CodeGeneration/Registration/ControllerDefinitionProvider.cs
@@ -614,7 +614,8 @@ private void CollectAuthorization(JToken templateNameValue, JObject authorizatio
             JToken targetValue = targetProperty.Value;
             SourceLocation targetLocation = targetValue.GetSourceInfo();
             AuthorizationBehavior authorizationBehavior = CreateActionDefinition(targetValue, targetLocation, explicitParameters, pathParameters, bodyParameters, requestBody: null, actionTargetDefinitionFactory: actionTarget => new AuthorizationBehavior(actionDefinition, actionTarget));
-            actionDefinition.Authorization.Add(authorizationBehavior);
+            if (authorizationBehavior != null)
+                actionDefinition.Authorization.Add(authorizationBehavior);
         }
 
         private JObject ApplyAuthorizationTemplate(JToken templateNameValue, JObject authorizationTemplateReference)

Original file line number	Diff line number	Diff line change
`@@ -5,5 +5,6 @@ internal static class SpecialHttpParameterName`
`5`	`5`	`public const string Body = "body";`
`6`	`6`	`public const string MediaType = "mediaType";`
`7`	`7`	`public const string FileName = "fileName";`
	`8`	`+ public const string Length = "length";`
`8`	`9`	`}`
`9`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,5 +6,6 @@ internal sealed class BodyParameterSource : ActionParameterSourceDefinition<Body`
`6`	`6`	`public const string RawPropertyName = "$RAW";`
`7`	`7`	`public const string MediaTypePropertyName = "$MEDIATYPE";`
`8`	`8`	`public const string FileNamePropertyName = "$FILENAME";`
	`9`	`+ public const string LengthPropertyName = "$LENGTH";`
`9`	`10`	`}`
`10`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,13 @@ public override void Resolve(IHttpParameterResolutionContext context)`
`32`	`32`	`break;`
`33`	`33`	`}`
`34`	`34`
	`35`	`+ case BodyParameterSource.LengthPropertyName:`
	`36`	`+ {`
	`37`	`+ Expression getBodyLengthCall = Expression.Call(context.RequestParameter, nameof(IHttpRequestDescriptor.GetBodyLength), Type.EmptyTypes);`
	`38`	`+ context.ResolveUsingValue(getBodyLengthCall);`
	`39`	`+ break;`
	`40`	`+ }`
	`41`	`+`
`35`	`42`	`default:`
`36`	`43`	`{`
`37`	`44`	`Type instanceType = context.ActionMetadata.SafeGetBodyContract();`
Original file line number	Diff line number	Diff line change
`@@ -614,7 +614,8 @@ private void CollectAuthorization(JToken templateNameValue, JObject authorizatio`
`614`	`614`	`JToken targetValue = targetProperty.Value;`
`615`	`615`	`SourceLocation targetLocation = targetValue.GetSourceInfo();`
`616`	`616`	`AuthorizationBehavior authorizationBehavior = CreateActionDefinition(targetValue, targetLocation, explicitParameters, pathParameters, bodyParameters, requestBody: null, actionTargetDefinitionFactory: actionTarget => new AuthorizationBehavior(actionDefinition, actionTarget));`
`617`		`- actionDefinition.Authorization.Add(authorizationBehavior);`
	`617`	`+ if (authorizationBehavior != null)`
	`618`	`+ actionDefinition.Authorization.Add(authorizationBehavior);`
`618`	`619`	`}`
`619`	`620`
`620`	`621`	`private JObject ApplyAuthorizationTemplate(JToken templateNameValue, JObject authorizationTemplateReference)`