Fixed bug which overwrote global handlers

Author: “lragji”

README.md

@@ -53,6 +53,9 @@ async function AppStartUp(rootRouter, DIContainer, application) {
         .get("/", (req, res) => {
             res.send(req.DIProp.fetchInstance("msg").message);
         })
+        .post("/", (req, res) => {
+            console.log(req.body);
+        })
         .get("/error", (req, res) => {
             throw new Error("This is an error to validate final 'ErrorResponseTransformer' error handling of library");
         });
@@ -64,8 +67,8 @@ async function AppStartUp(rootRouter, DIContainer, application) {
         .registerApplicationHandler(utilities.helmetMiddleware(), "*", 1, ApplicationTypes.Both)                                 //register helmet middleware for both application and health
         .registerApplicationHandler(utilities.bodyParserURLEncodingMiddleware(), "*", 2, ApplicationTypes.Main)                  //register body parser url middleware for application
         .registerApplicationHandler(utilities.bodyParserJSONEncodingMiddleware({ limit: '50M' }), "*", 3, ApplicationTypes.Main) //register body parser json middleware for application
-        .registerApplicationHandler(apiDocs.router, apiDocs.hostingPath, 3, ApplicationTypes.Main)                               //register api docs
-        .registerApplicationHandler(utilities.injectInRequestMiddleware("DIProp", DIContainer), "*", 4, ApplicationTypes.Main) //register DI container middleware
+        .registerApplicationHandler(apiDocs.router, apiDocs.hostingPath, 4, ApplicationTypes.Main)                               //register api docs
+        .registerApplicationHandler(utilities.injectInRequestMiddleware("DIProp", DIContainer), "*", 6, ApplicationTypes.Main) //register DI container middleware
         .overrideCatchAllErrorResponseTransformer((req, error) => ({                                                             //override the default catch all error response transformer
             path: req.path,
             status: 500,

examples/simple-express/index.mjs

@@ -16,6 +16,9 @@ async function AppStartUp(rootRouter, DIContainer, application) {
         .get("/", (req, res) => {
             res.send(req.DIProp.fetchInstance("msg").message);
         })
+        .post("/", (req, res) => {
+            console.log(req.body);
+        })
         .get("/error", (req, res) => {
             throw new Error("This is an error to validate final 'ErrorResponseTransformer' error handling of library");
         });
@@ -27,8 +30,8 @@ async function AppStartUp(rootRouter, DIContainer, application) {
         .registerApplicationHandler(utilities.helmetMiddleware(), "*", 1, ApplicationTypes.Both)                                 //register helmet middleware for both application and health
         .registerApplicationHandler(utilities.bodyParserURLEncodingMiddleware(), "*", 2, ApplicationTypes.Main)                  //register body parser url middleware for application
         .registerApplicationHandler(utilities.bodyParserJSONEncodingMiddleware({ limit: '50M' }), "*", 3, ApplicationTypes.Main) //register body parser json middleware for application
-        .registerApplicationHandler(apiDocs.router, apiDocs.hostingPath, 3, ApplicationTypes.Main)                               //register api docs
-        .registerApplicationHandler(utilities.injectInRequestMiddleware("DIProp", DIContainer), "*", 4, ApplicationTypes.Main)   //register DI container middleware
+        .registerApplicationHandler(apiDocs.router, apiDocs.hostingPath, 4, ApplicationTypes.Main)                               //register api docs
+        .registerApplicationHandler(utilities.injectInRequestMiddleware("DIProp", DIContainer), "*", 5, ApplicationTypes.Main)   //register DI container middleware
         .overrideCatchAllErrorResponseTransformer((req, error) => ({                                                             //override the default catch all error response transformer
             path: req.path,
             status: 500,

src/application-builder.ts

@@ -347,7 +347,7 @@ export class ApplicationBuilder {
     private setRoutes(map: SortedMap<ApplicationBuilderMiddleware | IRouter | SortedMap<ApplicationBuilderMiddleware>>, handler: ApplicationBuilderMiddleware | IRouter, hostingPath: HostingPath, order: number | undefined = undefined) {
         if (hostingPath === "*") {
             const existingMap = map.get(hostingPath) as SortedMap<ApplicationBuilderMiddleware> || new SortedMap<ApplicationBuilderMiddleware>();
-            existingMap.set(hostingPath, handler, order);
+            existingMap.set(`${hostingPath}-${order}`, handler, order);
             map.set(hostingPath, existingMap, 0);
         }
         else {