refactor(logger): change logger and logger decorator names

core/decorators/logger.ts → core/decorators/inject-logger.ts

@@ -1,17 +1,16 @@
-import { Reflect } from "../../di/shims/reflect.ts";
 import {
   LOGGER_NAMESPACE,
   LOGGING_FILTER,
   LOGGING_FORMATTER,
   LOGGING_PROVIDER,
 } from "../constants.ts";
 import { DiContainer, Type } from "../deps.ts";
-import { Log } from "../log.ts";
+import { Logger } from "../logger.ts";
 
 /**
  * Decorator used to inject a logger
  */
-export function Logger(
+export function InjectLogger(
   name?: string,
 ): PropertyDecorator & ParameterDecorator {
   return function (
@@ -20,7 +19,7 @@ export function Logger(
     propName?: string | symbol,
     paramIndex?: number,
   ) {
-    const logger = class extends Log {};
+    const logger = class extends Logger {};
     let loggerName = name;
 
     if (propName) {

core/log.ts → core/logger.ts

@@ -12,7 +12,7 @@ import { LoggingFormatter } from "./logging-formatter.ts";
 import { LoggingProvider } from "./logging-provider.ts";
 
 @Injectable()
-export class Log {
+export class Logger {
   @Optional()
   @Inject(LOGGER_NAMESPACE)
   namespace?: string;

core/mod.ts

@@ -13,7 +13,7 @@ export * from "./decorators/filter.ts";
 export * from "./decorators/get.ts";
 export * from "./decorators/head.ts";
 export * from "./decorators/header.ts";
-export * from "./decorators/logger.ts";
+export * from "./decorators/inject-logger.ts";
 export * from "./decorators/mv-module.ts";
 export * from "./decorators/param.ts";
 export * from "./decorators/patch.ts";
@@ -26,7 +26,7 @@ export * from "./decorators/transform.ts";
 export * from "./default-logging-formatter.ts";
 export * from "./lifecycle-events.ts";
 export * from "./log-level.ts";
-export * from "./log.ts";
+export * from "./logger.ts";
 export * from "./logging-filter.ts";
 export * from "./logging-formatter.ts";
 export * from "./logging-provider.ts";

core/platform.ts

@@ -12,7 +12,7 @@ import {
 } from "./controller-metadata.ts";
 import { DiContainer, Scope, Type, TypeIdentifier } from "./deps.ts";
 import { OnPlatformBootstrap } from "./lifecycle-events.ts";
-import { Log } from "./log.ts";
+import { Logger } from "./logger.ts";
 import { LoggingFilter } from "./logging-filter.ts";
 import { LoggingFormatter } from "./logging-formatter.ts";
 import { LoggingProvider } from "./logging-provider.ts";
@@ -49,7 +49,7 @@ export abstract class Platform {
   readonly #container: DiContainer;
   #module?: ModuleRef;
 
-  protected logger!: Log;
+  protected logger!: Logger;
 
   constructor(container: DiContainer) {
     this.#container = container;
@@ -117,7 +117,7 @@ export abstract class Platform {
       this.#container.createChild(moduleType.name),
       this.#diCache,
     );
-    this.logger = await this.resolve(Log);
+    this.logger = await this.resolve(Logger);
     this.logger.namespace = "Momentum";
     this.logger.loggerName = "Internal";
     this.logger.info(`Initializing platform`);

core/server-controller.ts

@@ -10,7 +10,7 @@ import { DependencyResolver, Scope, Type } from "./deps.ts";
 import { FilterCatalog } from "./filter-catalog.ts";
 import {
   ContentResult,
-  Log,
+  Logger,
   MvTransformer,
   OnRequestEnd,
   OnRequestStart,
@@ -35,13 +35,13 @@ export class ServerController {
   #middlewareCache?: MvMiddleware[];
   #globalErrorHandlers: { errorHandler: ErrorHandler; priority?: number }[] =
     [];
-  #logger!: Log;
+  #logger!: Logger;
 
   constructor(platform: ServerPlatform) {
     this.#platform = platform;
   }
 
-  async initialize(logger: Log) {
+  async initialize(logger: Logger) {
     this.#logger = logger;
     this.#platform.addMiddlewareHandler(async (context) => {
       const contextAccessor = new ContextAccessor(context, this.#platform);

platform-oak/platform-oak.ts

@@ -75,6 +75,14 @@ export class OakPlatform extends ServerPlatform {
 
   async postBootstrap() {
     this.#app.use(this.#router.routes());
+    this.#app.addEventListener("listen", ({ hostname, port, secure }) => {
+      this.logger.log([
+        `PlatformOak listening on `,
+        secure ? "https://" : "http://",
+        hostname ?? "localhost",
+        `:${port}`,
+      ].join(""));
+    });
     await super.postBootstrap();
   }
 
@@ -206,13 +214,7 @@ export class OakPlatform extends ServerPlatform {
    * Start listening for requests
    */
   async listen(options: ListenOptions) {
-    const listenPromise = this.#app.listen(options);
-    this.logger.info(
-      `PlatformOak listening on ${
-        options.hostname ? options.hostname + ":" : "port "
-      }${options.port}`,
-    );
-    await listenPromise;
+    await this.#app.listen(options);
   }
 
   private parseFormBody(form: URLSearchParams, identifier: string) {