@dooboostore/simple-boot-http-server 1.0.14 → 1.0.16

package/README.MD

@@ -198,23 +198,57 @@ const option = new HttpServerOption({
 
 Define global exception handlers in an `Advice` class. This class is managed by `@Sim` and can catch errors thrown from anywhere in the application.
 
+### Preventing `ERR_HTTP_HEADERS_SENT`
+
+A common issue in asynchronous frameworks is the `ERR_HTTP_HEADERS_SENT` error. This occurs when an error is thrown *after* a response has already been sent by another part of the application (like a filter). The global exception handler then tries to send a *second* response, causing a crash.
+
+To prevent this, **always check if a response has already been sent** before sending a new one from your exception handler. The `RequestResponse` object provides the `resIsDone()` method for this purpose.
+
 ```typescript
-import { Sim, ExceptionHandler } from '@dooboostore/simple-boot';
-import { RequestResponse, NotFoundError } from '@dooboostore/simple-boot-http-server';
+import { Sim, ExceptionHandler, Inject, ExceptionHandlerSituationType } from '@dooboostore/simple-boot';
+import { RequestResponse, NotFoundError, HttpError, InternalServerError, Mimes, HttpStatus } from '@dooboostore/simple-boot-http-server';
 
 @Sim
 export class GlobalErrorHandler {
+
     @ExceptionHandler({ type: NotFoundError })
     handleNotFound(rr: RequestResponse, e: NotFoundError) {
+        // Critical check
+        if (rr.resIsDone()) {
+            console.error(`Not found error for URL: ${rr.reqUrl}, but response already sent.`, e);
+            return;
+        }
         rr.resStatusCode(404).resSetHeader('X-Error-Message', 'Resource not found');
         console.error(`Not found error for URL: ${rr.reqUrl}`, e);
+        rr.resEnd(JSON.stringify({ error: 'Resource not found' }));
     }
 
     @ExceptionHandler()
-    handleAll(rr: RequestResponse, e: any) {
-        console.error('An unexpected error occurred:', e);
-        // Avoid sending detailed error in production
-        rr.resStatusCode(500).resWriteJson({ error: 'Internal Server Error' });
+    async catch(@Inject({situationType: ExceptionHandlerSituationType.ERROR_OBJECT}) e: any, rr: RequestResponse) {
+        console.error(`GlobalAdvice.catch ${rr.reqUrl}`, e);
+
+        // Critical check to prevent writing to an already-sent response
+        if (rr.resIsDone()) {
+            return;
+        }
+
+        rr.resStatusCode(HttpStatus.InternalServerError);
+        rr.resSetHeader('Content-Type', Mimes.ApplicationJson);
+
+        let data = '';
+        if (e instanceof HttpError) {
+            rr.resStatusCode(e.status);
+            data = JSON.stringify(e);
+        } else if (e instanceof Error){
+            const error = new InternalServerError();
+            error.data = {message: e.message, stack: e.stack}; // Be careful with stack in production
+            data = JSON.stringify(error);
+        } else {
+            const error = new InternalServerError();
+            error.data = e;
+            data = JSON.stringify(error);
+        }
+        await rr.resEnd(data);
     }
 }
 

package/SimpleBootHttpServer.d.ts

@@ -1,5 +1,3 @@
-/// <reference types="node" />
-/// <reference types="node" />
 import { SimpleApplication } from '@dooboostore/simple-boot/SimpleApplication';
 import { HttpServerOption } from './option/HttpServerOption';
 import { ConstructorType } from '@dooboostore/core/types';

package/SimpleBootHttpServer.js

@@ -18,7 +18,7 @@ import { HttpHeaders } from './codes/HttpHeaders';
 import { Mimes } from './codes/Mimes';
 import { ExceptionHandlerSituationType, targetExceptionHandler } from '@dooboostore/simple-boot/decorators/exception/ExceptionDecorator';
 import { getInject, SituationTypeContainer, SituationTypeContainers } from '@dooboostore/simple-boot/decorators/inject/Inject';
-import { ReflectUtils } from '@dooboostore/simple-boot/utils/reflect/ReflectUtils';
+import { ReflectUtils } from '@dooboostore/core/reflect/ReflectUtils';
 import { ReqFormUrlBody } from './models/datas/body/ReqFormUrlBody';
 import { ReqJsonBody } from './models/datas/body/ReqJsonBody';
 import { ReqHeader } from './models/datas/ReqHeader';

package/decorators/MethodMapping.js

@@ -1,5 +1,5 @@
 import 'reflect-metadata';
-import { ReflectUtils } from '@dooboostore/simple-boot/utils/reflect/ReflectUtils';
+import { ReflectUtils } from '@dooboostore/core/reflect/ReflectUtils';
 import { HttpMethod } from '../codes/HttpMethod';
 export var UrlMappingSituationType;
 (function (UrlMappingSituationType) {

package/endpoints/CrossDomainHeaderEndPoint.js

@@ -13,8 +13,8 @@ export class CrossDomainHeaderEndPoint {
         this.config = config;
     }
     endPoint(rr, app) {
-        var _a, _b, _c, _d;
         return __awaiter(this, void 0, void 0, function* () {
+            var _a, _b, _c, _d;
             if ((_a = this.config) === null || _a === void 0 ? void 0 : _a.accessControlAllowOrigin) {
                 rr.resSetHeader(HttpHeaders.AccessControlAllowOrigin, Array.isArray(this.config.accessControlAllowOrigin) ? this.config.accessControlAllowOrigin.join(', ') : this.config.accessControlAllowOrigin);
             }

package/filters/CacheFilter.js

@@ -22,11 +22,11 @@ export class CacheFilter {
         return __awaiter(this, void 0, void 0, function* () {
         });
     }
-    proceedBefore({ rr, app, carrier }) {
-        var _a, _b, _c, _d, _e, _f, _g;
-        var _h, _j, _k;
-        return __awaiter(this, void 0, void 0, function* () {
-            const filter = (_b = (_a = this.config).filter) === null || _b === void 0 ? void 0 : _b.call(_a, rr, app);
+    proceedBefore(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app, carrier }) {
+            var _b, _c, _d, _e, _f, _g, _h;
+            var _j, _k, _l;
+            const filter = (_c = (_b = this.config).filter) === null || _c === void 0 ? void 0 : _c.call(_b, rr, app);
             if (!filter) {
                 return true;
             }
@@ -41,16 +41,16 @@ export class CacheFilter {
                 }
             }
             const date = new Date().toISOString();
-            (_c = this.cacheConfig) !== null && _c !== void 0 ? _c : (this.cacheConfig = {
+            (_d = this.cacheConfig) !== null && _d !== void 0 ? _d : (this.cacheConfig = {
                 createDate: date,
                 updateDate: date,
                 caches: {}
             });
-            (_d = (_h = this.cacheConfig).createDate) !== null && _d !== void 0 ? _d : (_h.createDate = date);
-            (_e = (_j = this.cacheConfig).updateDate) !== null && _e !== void 0 ? _e : (_j.updateDate = date);
-            (_f = (_k = this.cacheConfig).caches) !== null && _f !== void 0 ? _f : (_k.caches = {});
+            (_e = (_j = this.cacheConfig).createDate) !== null && _e !== void 0 ? _e : (_j.createDate = date);
+            (_f = (_k = this.cacheConfig).updateDate) !== null && _f !== void 0 ? _f : (_k.updateDate = date);
+            (_g = (_l = this.cacheConfig).caches) !== null && _g !== void 0 ? _g : (_l.caches = {});
             const rkey = this.config.key(rr, app);
-            const sessionData = { key: rkey.key, lifeTime: (_g = rkey.lifeTime) !== null && _g !== void 0 ? _g : this.config.config.lifeTime, isUpdate: false, responseHeaders: rkey.responseHeaders };
+            const sessionData = { key: rkey.key, lifeTime: (_h = rkey.lifeTime) !== null && _h !== void 0 ? _h : this.config.config.lifeTime, isUpdate: false, responseHeaders: rkey.responseHeaders };
             const cache = this.cacheConfig.caches[sessionData.key];
             if (cache) {
                 const existes = this.config.config.cacheDir ? FileUtils.existesSync(cache.path) : this.storage.has(cache.path);
@@ -89,10 +89,10 @@ export class CacheFilter {
     getChunk(path) {
         return this.config.config.cacheDir ? FileUtils.readSync(path) : this.storage.get(path);
     }
-    proceedAfter({ rr, app, carrier }) {
-        var _a, _b;
-        var _c;
-        return __awaiter(this, void 0, void 0, function* () {
+    proceedAfter(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app, carrier }) {
+            var _b, _c;
+            var _d;
             const sessionData = carrier.get(CacheFilter.CARRIER_KEY_NAME);
             if (sessionData && sessionData.isUpdate) {
                 try {
@@ -108,10 +108,10 @@ export class CacheFilter {
                     p = this.writeChunk(destFileUUID, resBody);
                     const updateDateNow = Date.now();
                     const updateDate = new Date(updateDateNow).toISOString();
-                    (_a = (_c = this.cacheConfig).createDate) !== null && _a !== void 0 ? _a : (_c.createDate = updateDate);
+                    (_b = (_d = this.cacheConfig).createDate) !== null && _b !== void 0 ? _b : (_d.createDate = updateDate);
                     this.cacheConfig.updateDate = updateDate;
                     const responseHeaders = {};
-                    (_b = sessionData.responseHeaders) === null || _b === void 0 ? void 0 : _b.map(it => it.toLowerCase()).forEach(it => {
+                    (_c = sessionData.responseHeaders) === null || _c === void 0 ? void 0 : _c.map(it => it.toLowerCase()).forEach(it => {
                         responseHeaders[it] = rr.resHeader(it);
                     });
                     this.cacheConfig.caches[sessionData.key] = {

package/filters/HttpStatusFilter.js

@@ -11,15 +11,15 @@ export class HttpStatusFilter {
     constructor(httpStatus) {
         this.httpStatus = httpStatus;
     }
-    proceedBefore({ rr, app }) {
-        return __awaiter(this, void 0, void 0, function* () {
+    proceedBefore(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app }) {
             rr.resStatusCode(this.httpStatus);
             rr.resEnd();
             return false;
         });
     }
-    proceedAfter({ rr, app }) {
-        return __awaiter(this, void 0, void 0, function* () {
+    proceedAfter(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app }) {
             return true;
         });
     }

package/filters/OpenApi3Filter.js

@@ -25,8 +25,8 @@ export class OpenApi3Filter {
         return __awaiter(this, void 0, void 0, function* () {
         });
     }
-    proceedBefore({ rr, app }) {
-        return __awaiter(this, void 0, void 0, function* () {
+    proceedBefore(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app }) {
             if (rr.reqUrl === this.config.path) {
                 const map = app.routerManager.routingMap();
                 const data = {
@@ -48,8 +48,8 @@ export class OpenApi3Filter {
             return true;
         });
     }
-    proceedAfter({ rr, app }) {
-        return __awaiter(this, void 0, void 0, function* () {
+    proceedAfter(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app }) {
             return true;
         });
     }

package/filters/ResourceFilter.js

@@ -21,10 +21,10 @@ export class ResourceFilter {
         return __awaiter(this, void 0, void 0, function* () {
         });
     }
-    proceedBefore({ rr, app }) {
-        var _a;
-        return __awaiter(this, void 0, void 0, function* () {
-            const url = ((_a = rr.reqUrlPathName) !== null && _a !== void 0 ? _a : '').replace(/\.\./g, '');
+    proceedBefore(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app }) {
+            var _b;
+            const url = ((_b = rr.reqUrlPathName) !== null && _b !== void 0 ? _b : '').replace(/\.\./g, '');
             const urlObj = rr.reqUrlObject;
             let sw = true;
             const regExps = this.resourceRegex.filter(it => {
@@ -63,8 +63,8 @@ export class ResourceFilter {
             return sw;
         });
     }
-    proceedAfter({ rr, app }) {
-        return __awaiter(this, void 0, void 0, function* () {
+    proceedAfter(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app }) {
             return true;
         });
     }

package/filters/ThrowFilter.js

@@ -15,13 +15,13 @@ export class ThrowFilter {
         return __awaiter(this, void 0, void 0, function* () {
         });
     }
-    proceedBefore({ rr, app }) {
-        return __awaiter(this, void 0, void 0, function* () {
+    proceedBefore(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app }) {
             return true;
         });
     }
-    proceedAfter({ rr, app }) {
-        return __awaiter(this, void 0, void 0, function* () {
+    proceedAfter(_a) {
+        return __awaiter(this, arguments, void 0, function* ({ rr, app }) {
             const sw = rr.resIsDone();
             if (!sw) {
                 throw this.error;

package/models/RequestResponse.d.ts

@@ -1,8 +1,3 @@
-/// <reference types="node" />
-/// <reference types="node" />
-/// <reference types="node" />
-/// <reference types="node" />
-/// <reference types="node" />
 import { IncomingMessage, OutgoingHttpHeader, OutgoingHttpHeaders, ServerResponse } from 'http';
 import { HttpHeaders } from '../codes/HttpHeaders';
 import { Mimes } from '../codes/Mimes';
@@ -78,8 +73,8 @@ export declare class RequestResponse {
     reqSessionSet(key: string, value: any): void;
     reqSessionGet<T = any>(key: string): T | undefined;
     resSetStatusCode(statusCode: number): RequestResponseChain<number>;
-    resWrite(chunk: string | Buffer | any, encoding?: BufferEncoding): RequestResponseChain<string | Buffer>;
-    resWriteJson(chunk: any, encoding?: BufferEncoding): RequestResponseChain<string | Buffer>;
+    resWrite(chunk: string | Buffer | any, encoding?: BufferEncoding): RequestResponseChain<string | Buffer<ArrayBufferLike>>;
+    resWriteJson(chunk: any, encoding?: BufferEncoding): RequestResponseChain<string | Buffer<ArrayBufferLike>>;
     resSetHeader(key: HttpHeaders | string, value: string | string[]): RequestResponseChain<ServerResponse<IncomingMessage>>;
     resAddHeader(key: HttpHeaders | string, value: string | string[]): RequestResponseChain<ServerResponse<IncomingMessage>>;
     resSetHeaders(headers: {

package/models/datas/MultipartData.d.ts

@@ -1,5 +1,3 @@
-/// <reference types="node" />
-/// <reference types="node" />
 export type MultipartData = {
     isFile: true;
     name: string;

package/models/datas/ReqHeader.d.ts

@@ -1,4 +1,3 @@
-/// <reference types="node" />
 import { IncomingHttpHeaders } from 'http';
 export declare class ReqHeader implements IncomingHttpHeaders {
     [key: string]: string | string[] | undefined;

package/option/HttpServerOption.d.ts

@@ -1,5 +1,3 @@
-/// <reference types="node" />
-/// <reference types="node" />
 import { InitOptionType, SimOption } from '@dooboostore/simple-boot/SimOption';
 import { ConstructorType } from '@dooboostore/core/types';
 import { Server as HttpServer, ServerOptions } from 'http';

package/package.json

@@ -1,7 +1,8 @@
 {
   "name": "@dooboostore/simple-boot-http-server",
-  "version": "1.0.14",
-  "main": "dist/SimpleBootHttpServer.js",
+  "version": "1.0.16",
+  "main": "dist/index.js",
+  "types": "dist/index.d.ts",
   "license": "MIT",
   "description": "back end http server frameworks",
   "files": [
@@ -51,7 +52,7 @@
     "ts-node": "^10.0.0",
     "ts-node-dev": "^1.1.8",
     "tsconfig-paths": "^3.9.0",
-    "typescript": "^4.4.3"
+    "typescript": "^5.9.2"
   },
   "dependencies": {
     "@types/node-gzip": "^1.1.0",
@@ -59,17 +60,16 @@
     "mime-types": "^2.1.34",
     "node-gzip": "^1.1.2",
     "reflect-metadata": "^0.2.2",
-    "canvas": "^3.1.0",
-    "@dooboostore/core": "1.0.7",
+    "@dooboostore/core": "1.0.8",
+    "@dooboostore/dom-render": "1.0.102",
     "@dooboostore/core-node": "1.0.1",
     "@dooboostore/core-web": "1.0.1",
-    "@dooboostore/dom-render": "1.0.99",
-    "@dooboostore/simple-boot": "1.0.43"
+    "@dooboostore/simple-boot": "1.0.44"
   },
   "scripts": {
-    "build": "rm -rf dist && tsc -p tsconfig.build.json",
+    "tsc:build": "rm -rf dist && tsc -p tsconfig.build.json",
     "clean": "rm -rf  dist",
-    "npm:build": "pnpm run build && cp README.MD dist/ && pnpm pack && TARBALL_NAME=$(ls dooboostore-simple-boot-http-server-*.tgz | head -n 1) && tar -xzf \"$TARBALL_NAME\" -C dist --strip-components=1 package/package.json && rm \"$TARBALL_NAME\"",
+    "npm:build": "pnpm run clean && pnpm run tsc:build && cp README.MD dist/ && pnpm pack && TARBALL_NAME=$(ls dooboostore-simple-boot-http-server-*.tgz | head -n 1) && tar -xzf \"$TARBALL_NAME\" -C dist --strip-components=1 package/package.json && rm \"$TARBALL_NAME\"",
     "npm:clean": "rm -rf  dist",
     "npm:publish": "pnpm run npm:build && cd dist && pnpm publish --no-git-checks",
     "npm:publish:clean": "pnpm run npm:publish && rm -rf dist",

package/session/SessionManager.js

@@ -25,8 +25,8 @@ export class SessionManager {
         }), this.sessionOption.expiredTime);
     }
     getSessionUUIDs() {
-        var _a;
         return __awaiter(this, void 0, void 0, function* () {
+            var _a;
             if ((_a = this.sessionOption.provider) === null || _a === void 0 ? void 0 : _a.uuids) {
                 return this.sessionOption.provider.uuids();
             }
@@ -36,8 +36,8 @@ export class SessionManager {
         });
     }
     deleteSession(uuid) {
-        var _a;
         return __awaiter(this, void 0, void 0, function* () {
+            var _a;
             if ((_a = this.sessionOption.provider) === null || _a === void 0 ? void 0 : _a.delete) {
                 this.sessionOption.provider.delete(uuid);
             }
@@ -47,8 +47,8 @@ export class SessionManager {
         });
     }
     getSessionDataSet(uuid) {
-        var _a;
         return __awaiter(this, void 0, void 0, function* () {
+            var _a;
             if ((_a = this.sessionOption.provider) === null || _a === void 0 ? void 0 : _a.get) {
                 return this.sessionOption.provider.get(uuid);
             }
@@ -58,8 +58,8 @@ export class SessionManager {
         });
     }
     setSessionDataSet(uuid, dataSet) {
-        var _a;
         return __awaiter(this, void 0, void 0, function* () {
+            var _a;
             if ((_a = this.sessionOption.provider) === null || _a === void 0 ? void 0 : _a.set) {
                 this.sessionOption.provider.set(uuid, dataSet);
             }
@@ -75,8 +75,8 @@ export class SessionManager {
         return RandomUtils.uuid('xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx');
     }
     session(rrOrUUID) {
-        var _a;
         return __awaiter(this, void 0, void 0, function* () {
+            var _a;
             let uuid;
             if (rrOrUUID instanceof RequestResponse) {
                 uuid = (_a = rrOrUUID.reqCookieGet(this.sessionOption.key)) !== null && _a !== void 0 ? _a : this.makeUUID();