Merge pull request #7 from Defirence/Defirence/issue5

Fix #5

Author: Defirence

boiler-plate/node-express-server/.gitignore

@@ -0,0 +1 @@
+node_modules/

boiler-plate/node-express-server/README.md

@@ -1,9 +1,9 @@
 # node-express-server
 A simple node.js webserver with basic API functionality, written in pure JavaScript.
 Dependencies:
-- yarn version: 1.22.19
-- npm version: 8.15.0
-- node.js version: v16.17.0 LTS
+- yarn version: 1.22.22
+- npm version: 10.9.2
+- node.js version: v22.14.0
 - ECMA version: `latest`, other information present in `.eslintrc.json`
 
 Makes use of Snyk for IaC and Security scanning.

boiler-plate/node-express-server/package.json

@@ -7,7 +7,9 @@
   "author": "Defirence",
   "license": "CC0-Universal",
   "dependencies": {
-    "eslint": ">=5.16.0"
+    "eslint": ">=5.16.0",
+    "express": "^5.1.0",
+    "user-agents": "^1.1.502"
   },
   "devDependencies": {
     "eslint-config-google": "^0.14.0"

boiler-plate/node-express-server/webserver.js

@@ -1,9 +1,8 @@
 const express = require('express');
 const app = express();
 const port = 3000;
-// user-agents DOES NOT READ the actual userAgent string.
-// it just generates random userAgent strings.
-let userAgent = require('user-agents');
+const UserAgents = require('user-agents');
+const userAgent = new UserAgents().toString();
 const dir = '/user';
 
 app.get('/', (req, res) => {
@@ -38,6 +37,5 @@ app.use(() => {
   console.log(userAgent);
 });
 
-// userAgentLogging
-userAgent = new userAgent(); // eslint new-cap: "off"
-console.log(userAgent.toString()); // eslint new-cap: "off"
+// userAgentLogging - fix this to not use new userAgent(); and rather pull the existing UA string.
+console.log(userAgent); // Logging the existing userAgent string