Bug / Lazy Error

.dockerignore

@@ -0,0 +1,19 @@
+**/.dockerignore
+**/.env
+**/.git
+**/.gitignore
+**/.github
+**/.vs
+**/.vscode
+**/.project
+**/*.*proj.user
+**/.settings
+**/.toolstarget
+**/Dockerfile*
+**/bin
+**/obj
+**/*.swp
+**/*.md
+docs/**/*
+research/**/*
+samples/**/*

docs/features/authorization.md

@@ -46,3 +46,11 @@ and this handler directly throws;
 - `UnauthorizedAccessExcetpin` when forbidden
 
 instead of using `IAuthenticationHandler` fallback methods.
+
+## Disabled
+
+You can disable this feature by calling `Disabled()` method;
+
+```csharp
+c => c.Disabled()
+```

src/recipe/Baked.Recipe.Service.Application/Authentication/AuthenticationConfigurator.cs

@@ -1,9 +1,3 @@
-using Baked.Architecture;
+namespace Baked.Authentication;
 
-namespace Baked.Authentication;
-
-public class AuthenticationConfigurator
-{
-    public IFeature<AuthenticationConfigurator> Disabled() =>
-        Feature.Empty<AuthenticationConfigurator>();
-}
+public class AuthenticationConfigurator { }

src/recipe/Baked.Recipe.Service.Application/Orm/AutoMap/AutoMapOrmFeature.cs

@@ -1,4 +1,5 @@
 using Baked.Architecture;
+using Baked.RestApi;
 using Baked.RestApi.Conventions;
 using FluentNHibernate.Conventions.Helpers;
 using FluentNHibernate.Mapping;
@@ -8,6 +9,7 @@
 using Microsoft.Extensions.Logging;
 using NHibernate;
 using NHibernate.Exceptions;
+using NHibernate.Proxy;
 
 namespace Baked.Orm.AutoMap;
 
@@ -79,6 +81,13 @@ public void Configure(LayerConfigurator configurator)
             });
         });
 
+        configurator.ConfigureMvcNewtonsoftJsonOptions(options =>
+        {
+            if (options.SerializerSettings.ContractResolver is null) { return; }
+
+            options.SerializerSettings.ContractResolver = new ProxyAwareContractResolver<INHibernateProxy>(options.SerializerSettings.ContractResolver);
+        });
+
         configurator.ConfigureApiModelConventions(conventions =>
         {
             var domainModel = configurator.Context.GetDomainModel();

src/recipe/Baked.Recipe.Service.Application/RestApi/ProxyAwareContractResolver.cs

@@ -0,0 +1,17 @@
+using Newtonsoft.Json.Serialization;
+
+namespace Baked.RestApi;
+
+public class ProxyAwareContractResolver<TProxyInterface>(IContractResolver _real)
+    : IContractResolver
+{
+    public JsonContract ResolveContract(Type type)
+    {
+        if (type.IsAssignableTo(typeof(TProxyInterface)))
+        {
+            type = type.BaseType ?? throw new($"Proxy type {type.FullName} should have a base type!!");
+        }
+
+        return _real.ResolveContract(type);
+    }
+}

test/recipe/Baked.Test.Recipe.Service.Application/Dockerfile

@@ -1,3 +1,5 @@
+# syntax=docker/dockerfile:1.7-labs
+
 FROM mcr.microsoft.com/dotnet/aspnet:8.0 AS base
 
 ARG ENVIRONMENT=Production
@@ -8,6 +10,9 @@ ENV ASPNETCORE_URLS=http://+:80
 
 FROM mcr.microsoft.com/dotnet/sdk:8.0 AS publish
 
+COPY --exclude=**/*.cs --exclude=**/*.json . .
+RUN dotnet restore
+
 COPY . .
 RUN dotnet publish ./test/recipe/Baked.Test.Recipe.Service.Application -c Release -o /app/publish
 

test/recipe/Baked.Test.Recipe.Service.Test/Orm/LazyLoading.cs

@@ -0,0 +1,21 @@
+using Newtonsoft.Json;
+using System.Net;
+using System.Net.Http.Json;
+
+namespace Baked.Test.Orm;
+
+public class LazyLoading : TestServiceNfr
+{
+    [Test]
+    public async Task Proxy_classes_serialized_correctly()
+    {
+        var parentResponse = await Client.PostAsync("/parents", JsonContent.Create(new { name = "test" }));
+        dynamic? parent = JsonConvert.DeserializeObject(await parentResponse.Content.ReadAsStringAsync());
+
+        await Client.PostAsync($"/parents/{parent?.id}/children", JsonContent.Create(new { }));
+
+        var response = await Client.GetAsync($"/children");
+
+        response.StatusCode.ShouldBe(HttpStatusCode.OK);
+    }
+}

test/recipe/Baked.Test.Recipe.Service/Orm/Child.cs

@@ -24,4 +24,7 @@ public class Children(IQueryContext<Child> _context)
 {
     internal List<Child> ByParent(Parent parent) =>
         _context.By(e => e.Parent == parent);
+
+    public List<Child> By() =>
+        _context.By(c => true);
 }

unreleased.md

@@ -6,3 +6,8 @@
   - Root namespace is `Baked`
   - `Forge` is now `Bake`
   - `Bluprints` are now `Recipes`
+- `Authentication.Disabled` was removed
+
+## Bugfixes
+
+- NHibernate proxies were causing serialization error, fixed