Merge pull request #132 from neonexus/master

Added `ignoreAssets`. Rebuilt package-lock.json.

Author: neonexus

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+# [v5.3.3](https://github.com/neonexus/sails-react-bootstrap-webpack/compare/v5.3.2...v5.3.3) (2024-04-29)
+### Features
+
+* Added `ignoreAssets` option to the request logger. Should cut down noise a bit.
+* Re-installed dependencies to do a package-lock reset. Should fix issues reported by Snyk.
+
 # [v5.3.2](https://github.com/neonexus/sails-react-bootstrap-webpack/compare/v5.3.1...v5.3.2) (2024-04-22)
 ### Features
 

README.md

@@ -225,6 +225,14 @@ overridden in the `config/local.js`, just like every other option. If the option
                 <br /><br />See <a href="#request-logging">Request Logging</a> for more info.
             </td>
         </tr>
+        <tr>
+            <td><code>log.ignoreAssets</code></td>
+            <td><a href="config/log.js"><code>log.js</code></a></td>
+            <td><code>true</code></td>
+            <td>
+                When enabled (and `captureRequests` is `true`), this will force the logger to skip over assets (things like `.js` / `.css`, etc.).
+            </td>
+        </tr>
         <tr>
             <td><code>models.validateOnBootstrap</code></td>
             <td><a href="config/models.js"><code>models.js</code></a></td>

api/helpers/finalize-request-log.js

@@ -40,7 +40,7 @@ module.exports = {
     },
 
     fn: async (inputs, exits) => {
-        if (inputs.req.id && sails.config.log.captureRequests === true) {
+        if (inputs.req.id && inputs.req.id !== 'IGNORE' && sails.config.log.captureRequests === true) {
             const bleep = '*******';
             let out = _.merge({}, inputs.body),
                 headers = _.merge({}, inputs.res.getHeaders()); // copy the object

api/hooks/request-logger.js

@@ -9,7 +9,19 @@ module.exports = (sails) => {
         routes: {
             before: {
                 '*': function(req, res, next) {
-                    if (req.method !== 'HEAD' && req.path !== '/__getcookie' && req.path !== '/' && sails.config.log.captureRequests === true) {
+                    if (
+                        req.method !== 'HEAD'
+                        && req.path !== '/__getcookie'
+                        && req.path !== '/_ping'
+                        && req.path !== '/'
+                        && sails.config.log.captureRequests === true
+                    ) {
+                        if (sails.config.log.ignoreAssets && req.path.includes('.')) {
+                            req.id = 'IGNORE';
+
+                            return next();
+                        }
+
                         const bleep = '*******';
 
                         let body = _.merge({}, req.body),

config/log.js

@@ -38,5 +38,13 @@ module.exports.log = {
      *                                                                  *
      ********************************************************************/
 
-    captureRequests: true
+    captureRequests: true,
+
+    /********************************************************************
+     * This will ignore logging of asset requests                       *
+     * (things like `.js`, `.css`, etc.), when capturing request data.  *
+     *                                                                  *
+     * Turning this off could make request logs balloon very quickly.   *
+     ********************************************************************/
+    ignoreAssets: true
 };