Merge pull request #15 from scimmyjs/issue/13-pagination-constraint-p…

…arsing

Add: casting of pagination query parameters from strings to numbers

src/routers.js

@@ -75,7 +75,7 @@ export default class SCIMMYRouters extends Router {
         // Make sure SCIM JSON is decoded in request body
         this.use(express.json({type: "application/scim+json", limit: SCIMMY.Config.get()?.bulk?.maxPayloadSize ?? "1mb"}));
 
-        // Listen for first request to determine basepath for all resource types
+        // Listen for incoming requests to determine basepath for all resource types
         this.use("/", (req, res, next) => {
             res.setHeader("Content-Type", "application/scim+json");
             SCIMMY.Resources.Schema.basepath(req.baseUrl);
@@ -99,6 +99,18 @@ export default class SCIMMYRouters extends Router {
             }
         });
 
+        // Cast pagination query parameters from strings to numbers...
+        this.use(({query}, res, next) => {
+            for (let param of ["startIndex", "count"]) {
+                // ...but only if they were defined, were strings, and are valid as numbers
+                if (!!query[param] && typeof query[param] === "string" && !Number.isNaN(+query[param])) {
+                    query[param] = +query[param];
+                }
+            }
+
+            next();
+        });
+
         // Register core service provider endpoints
         this.use(new Schemas());
         this.use(new ResourceTypes());