@compas/server - npm Package Compare versions

Comparing version 0.0.166 to 0.0.167

package.json

 {
   "name": "@compas/server",
-  "version": "0.0.166",
+  "version": "0.0.167",
   "description": "Koa server and common middleware",
@@ -18,3 +18,3 @@ "main": "./index.js",
   "dependencies": {
-    "@compas/stdlib": "0.0.166",
+    "@compas/stdlib": "0.0.167",
     "@types/formidable": "1.2.4",
@@ -21,0 +21,0 @@ "@types/koa": "2.13.4",

src/app.js

@@ -85,3 +85,3 @@ import { isProduction } from "@compas/stdlib";
 
-  app.use(logMiddleware(opts.logOptions ?? {}));
+  app.use(logMiddleware(app, opts.logOptions ?? {}));
   app.use(errorHandler(opts.errorOptions ?? {}));
@@ -88,0 +88,0 @@ app.use(notFoundHandler());

src/middleware/log.d.ts

 /**
  * Log basic request and response information
  *
+ * @param {import("koa")} app
  * @param {{ disableRootEvent?: boolean }} options
  */
-export function logMiddleware(options: {
+export function logMiddleware(app: import("koa"), options: {
     disableRootEvent?: boolean;
 }): (ctx: any, next: any) => Promise<void>;
 //# sourceMappingURL=log.d.ts.map

src/middleware/log.js

 import { Transform } from "stream";
 import {
+  AppError,
   eventStart,
@@ -14,5 +15,6 @@ eventStop,
  *
+ * @param {import("koa")} app
  * @param {{ disableRootEvent?: boolean }} options
  */
-export function logMiddleware(options) {
+export function logMiddleware(app, options) {
   /**
@@ -50,2 +52,17 @@ * Real log function
 
+  // Log stream errors after the headers are sent
+  const logger = newLogger({
+    ctx: {
+      type: "http-error",
+    },
+  });
+  app.on("error", (error, ctx) => {
+    (ctx?.log ?? logger).info({
+      type: "http-error",
+      headerSent: error.headerSent,
+      syscall: error.syscall,
+      error: AppError.format(error),
+    });
+  });
+
   return async (ctx, next) => {
@@ -52,0 +69,0 @@ const startTime = process.hrtime.bigint();

src/middleware/sendFile.js

@@ -64,2 +64,3 @@ import { isNil } from "@compas/stdlib";
       if (isNil(start) || start > file.contentLength) {
+        // '-500' results in the last 500 bytes send
         start = file.contentLength - end;
@@ -66,0 +67,0 @@ end = file.contentLength - 1;

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

48669

increased by1.07%

Lines of code

1256

increased by1.45%

Dependency changes

• + Added@compas/stdlib@0.0.167(transitive)

• - Removed@compas/stdlib@0.0.166(transitive)

• Updated@compas/stdlib@0.0.167