Added logging to reflection generation of APIs. Fixes #36

Author: csharpfritz

Fritz.InstantAPIs/MapApiExtensions.cs

@@ -4,6 +4,7 @@
 using Microsoft.AspNetCore.Http;
 using System.ComponentModel.DataAnnotations;
 using System.Reflection;
+using Microsoft.Extensions.Logging;
 
 namespace Fritz.InstantAPIs;
 
@@ -12,12 +13,14 @@ internal class MapApiExtensions
 
 	// TODO: Authentication / Authorization
 	private static Dictionary<Type, PropertyInfo> _IdLookup = new();
+	private static ILogger Logger;
 
-	internal static void Initialize<D,C>() 
+	internal static void Initialize<D,C>(ILogger logger) 
 		where D: DbContext
 		where C: class 
 	{
 
+		Logger = logger;
 		var theType = typeof(C);
 		var idProp = theType.GetProperty("id", BindingFlags.IgnoreCase | BindingFlags.Public | BindingFlags.Instance) ?? theType.GetProperties().FirstOrDefault(p => p.CustomAttributes.Any(a => a.AttributeType == typeof(KeyAttribute)));
 
@@ -33,6 +36,7 @@ internal static void MapInstantGetAll<D, C>(IEndpointRouteBuilder app, string ur
 		where D : DbContext where C : class
 	{
 
+		Logger.LogInformation($"Created API: HTTP GET\t{url}");
 		app.MapGet(url, ([FromServices] D db) =>
 		{
 			return Results.Ok(db.Set<C>());
@@ -51,6 +55,8 @@ internal static void MapGetById<D,C>(IEndpointRouteBuilder app, string url)
 
 		if (idProp == null) return;
 
+		Logger.LogInformation($"Created API: HTTP GET\t{url}/{{id}}");
+
 		app.MapGet($"{url}/{{id}}", async ([FromServices] D db, [FromRoute] string id) =>
 		{
 
@@ -76,6 +82,7 @@ internal static void MapInstantPost<D, C>(IEndpointRouteBuilder app, string url)
 		where D : DbContext where C : class
 	{
 
+		Logger.LogInformation($"Created API: HTTP POST\t{url}");
 
 		app.MapPost(url, async ([FromServices] D db, [FromBody] C newObj) =>
 		{
@@ -92,6 +99,7 @@ internal static void MapInstantPut<D, C>(IEndpointRouteBuilder app, string url)
 		where D : DbContext where C : class
 	{
 
+		Logger.LogInformation($"Created API: HTTP PUT\t{url}");
 
 		app.MapPut($"{url}/{{id}}", async ([FromServices] D db, [FromRoute] string id, [FromBody] C newObj) =>
 		{
@@ -113,6 +121,7 @@ internal static void MapDeleteById<D, C>(IEndpointRouteBuilder app, string url)
 		var idProp = _IdLookup[theType];
 
 		if (idProp == null) return;
+		Logger.LogInformation($"Created API: HTTP DELETE\t{url}");
 
 		app.MapDelete($"{url}/{{id}}", async ([FromServices] D db, [FromRoute] string id) =>
 		{

Fritz.InstantAPIs/WebApplicationExtensions.cs

@@ -1,13 +1,16 @@
 using Microsoft.AspNetCore.Routing;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
+using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Options;
 using System.Reflection;
 
 namespace Microsoft.AspNetCore.Builder;
 
 public static class WebApplicationExtensions
 {
+	
+	internal const string LOGGER_CATEGORY_NAME = "InstantAPI";
 
 	private static InstantAPIsConfig Configuration { get; set; } = new();
 
@@ -33,6 +36,9 @@ public static IEndpointRouteBuilder MapInstantAPIs<D>(this IEndpointRouteBuilder
 	private static void MapInstantAPIsUsingReflection<D>(IEndpointRouteBuilder app, IEnumerable<TypeTable> requestedTables) where D : DbContext
 	{
 
+		var loggerFactory = app.ServiceProvider.GetRequiredService<ILoggerFactory>();
+		var logger = loggerFactory.CreateLogger(LOGGER_CATEGORY_NAME);
+
 		var allMethods = typeof(MapApiExtensions).GetMethods(BindingFlags.NonPublic | BindingFlags.Static).Where(m => m.Name.StartsWith("Map")).ToArray();
 		var initialize = typeof(MapApiExtensions).GetMethod("Initialize", BindingFlags.NonPublic | BindingFlags.Static);
 		foreach (var table in requestedTables)
@@ -41,7 +47,7 @@ private static void MapInstantAPIsUsingReflection<D>(IEndpointRouteBuilder app,
 			// The default URL for an InstantAPI is /api/TABLENAME
 			var url = $"/api/{table.Name}";
 
-			initialize.MakeGenericMethod(typeof(D), table.InstanceType).Invoke(null, null);
+			initialize.MakeGenericMethod(typeof(D), table.InstanceType).Invoke(null, new[] { logger });
 
 			// The remaining private static methods in this class build out the Mapped API methods..
 			// let's use some reflection to get them

WorkingApi/MyContext.cs

@@ -4,10 +4,6 @@ namespace WorkingApi;
 
 public class MyContext : DbContext 
 {
-	public MyContext()
-	{
-
-	}
 	public MyContext(DbContextOptions<MyContext> options) : base(options) {}
 
 	public DbSet<Contact> Contacts => Set<Contact>();

WorkingApi/Program.cs

@@ -7,32 +7,15 @@
 var builder = WebApplication.CreateBuilder(args);
 
 builder.Services.AddSqlite<MyContext>("Data Source=contacts.db");
-//builder.Services.AddEndpointsApiExplorer();
-//builder.Services.AddSwaggerGen();
-builder.Services.AddInstantAPIs(options =>
-{
-    options.EnableSwagger = Fritz.InstantAPIs.EnableSwagger.DevelopmentOnly;
-    options.Swagger = config =>
-    {
-        config.SwaggerDoc("v1", new OpenApiInfo
-        {
-            Title = "My Working API",
-            Description = "An ASP.NET Core Web API"
-        });
-    };
-});
+builder.Services.AddInstantAPIs();
 
 var app = builder.Build();
 
 var sw = Stopwatch.StartNew();
 
-app.MapInstantAPIs<MyContext>(options =>
+app.MapInstantAPIs<MyContext>(config =>
 {
-	options.IncludeTable(db => db.Contacts, (ApiMethodsToGenerate.GetById | ApiMethodsToGenerate.Get));
+	config.IncludeTable(db => db.Contacts, ApiMethodsToGenerate.All);
 });
-Console.WriteLine($"Elapsed time to build InstantAPIs: {sw.Elapsed}");
-
-//app.UseSwagger();
-//app.UseSwaggerUI();
 
 app.Run();