@nestjs/platform-fastify 12.1.0 → 12.1.1

package/adapters/fastify-adapter.d.ts

@@ -122,8 +122,8 @@ export declare class FastifyAdapter<TServer extends RawServerBase = RawServerDef
     inject(opts: InjectOptions | string): Promise<LightMyRequestResponse>;
     close(): Promise<void>;
     initHttpServer(options?: NestApplicationOptions): void;
-    useStaticAssets(options: FastifyStaticOptions): Promise<any>;
-    setViewEngine(options: FastifyViewOptions | string): Promise<any>;
+    useStaticAssets(options: FastifyStaticOptions): any;
+    setViewEngine(options: FastifyViewOptions | string): any;
     isHeadersSent(response: TReply): boolean;
     getHeader(response: any, name: string): any;
     setHeader(response: TReply, name: string, value: string): FastifyReply<RouteGenericInterface, TServer, TRawRequest, TRawResponse, unknown, import("fastify").FastifySchema, import("fastify").FastifyTypeProviderDefault, unknown>;

package/adapters/fastify-adapter.js

@@ -280,8 +280,9 @@ export class FastifyAdapter extends AbstractHttpAdapter {
             }
             body = body.getStream();
         }
-        if (fastifyReply.getHeader('Content-Type') !== undefined &&
-            fastifyReply.getHeader('Content-Type') !== 'application/json' &&
+        const responseContentType = fastifyReply.getHeader('Content-Type');
+        if (typeof responseContentType === 'string' &&
+            !responseContentType.startsWith('application/json') &&
             body?.statusCode >= HttpStatus.BAD_REQUEST) {
             Logger.warn("Content-Type doesn't match Reply body, you might need a custom ExceptionFilter for non-JSON responses", FastifyAdapter.name);
             fastifyReply.header('Content-Type', 'application/json');
@@ -392,15 +393,20 @@ export class FastifyAdapter extends AbstractHttpAdapter {
             this.trackOpenConnections();
         }
     }
-    async useStaticAssets(options) {
-        return this.register(await loadPackage('@fastify/static', 'FastifyAdapter.useStaticAssets()', () => import('@fastify/static')), options);
+    // `register()` accepts a promise of a plugin, so the import is handed over
+    // unawaited on purpose. `NestApplication.useStaticAssets()` discards what
+    // this returns, so awaiting here would enqueue the plugin after the caller
+    // has already moved on to `listen()`.
+    useStaticAssets(options) {
+        return this.register(loadPackage('@fastify/static', 'FastifyAdapter.useStaticAssets()', () => import('@fastify/static')), options);
     }
-    async setViewEngine(options) {
+    // Handed over unawaited for the same reason as `useStaticAssets()` above.
+    setViewEngine(options) {
         if (isString(options)) {
             new Logger('FastifyAdapter').error("setViewEngine() doesn't support a string argument.");
             process.exit(1);
         }
-        return this.register(await loadPackage('@fastify/view', 'FastifyAdapter.setViewEngine()', () => import('@fastify/view')), options);
+        return this.register(loadPackage('@fastify/view', 'FastifyAdapter.setViewEngine()', () => import('@fastify/view')), options);
     }
     isHeadersSent(response) {
         return response.sent;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/platform-fastify",
-  "version": "12.1.0",
+  "version": "12.1.1",
   "description": "Nest - modern, fast, powerful node.js web framework (@platform-fastify)",
   "author": "Kamil Mysliwiec",
   "license": "MIT",
@@ -54,5 +54,5 @@
       "optional": true
     }
   },
-  "gitHead": "6739656fa3e8c98bd55883f0363a2254e6412232"
+  "gitHead": "9feedffde8ef8c86f0cd521153740575dc579d88"
 }