reaname hook, add comments

fleetdm · Mar 22, 2024 · acda3d7 · acda3d7
1 parent 5d535fa
commit acda3d7
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...lity-dashboard/api/hooks/OktaSSO/index.js → ...ity-dashboard/api/hooks/okta-sso/index.js b/...lity-dashboard/api/hooks/OktaSSO/index.js → ...ity-dashboard/api/hooks/okta-sso/index.js
@@ -35,6 +35,7 @@ module.exports = function (sails){
       }
 
       // Clone the existing routes
+      // NOTE: Changing sails.config after the app lifts goes against Sails.js conventions and this code should not be reproduced.
       let appRoutes = Object.assign({}, sails.config.routes);
       // Remove the routes for the built-in login page..
       delete appRoutes['GET /login'];
@@ -45,7 +46,7 @@ module.exports = function (sails){
         'bodyParser',
         'compress',
         'poweredBy',
-        'www',
+        'www',// Note: This changes the conventions of Sails.js. Don't ever replicate this or use Passport.js.
         'oktaSSO',
         'router',
         'favicon',