refactor(service): users/auth: add todo tag comments for removing leg…

…acy authentication configuration model references
ngageoint · Sep 13, 2024 · 17a80ce · 17a80ce
1 parent 21b9d7f
commit 17a80ce
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 5 deletions.
diff --git a/service/src/provision/index.js b/service/src/provision/index.js
@@ -1,5 +1,6 @@
 const { modulesPathsInDir } = require('../utilities/loader');
 const log = require('../logger');
+// TODO: users-next
 const AuthenticationConfiguration = require('../models/authenticationconfiguration');
 
 /**

diff --git a/service/src/routes/authenticationconfigurations.js b/service/src/routes/authenticationconfigurations.js
@@ -4,6 +4,7 @@ const log = require('winston')
   , AsyncLock = require('async-lock')
   , access = require('../access')
   , Authentication = require('../models/authentication')
+  // TODO: users-next
   , AuthenticationConfiguration = require('../models/authenticationconfiguration')
   , AuthenticationConfigurationTransformer = require('../transformers/authenticationconfiguration')
   , SecretStoreService = require('../security/secret-store-service')

diff --git a/service/src/routes/setup.js b/service/src/routes/setup.js
@@ -4,6 +4,7 @@ module.exports = function (app, security) {
     , User = require('../models/user')
     , Device = require('../models/device')
     , userTransformer = require('../transformers/user')
+    // TODO: users-next
     , AuthenticationConfiguration = require('../models/authenticationconfiguration');
 
   function authorizeSetup(req, res, next) {

diff --git a/service/src/security/utilities/secure-property-appender.js b/service/src/security/utilities/secure-property-appender.js
@@ -1,11 +1,12 @@
 'use strict';
 
 const SecretStoreService = require('../secret-store-service')
+// TODO: users-next
     , AuthenticationConfiguration = require('../../models/authenticationconfiguration')
 
 /**
  * Helper function to append secure properties to a configuration under the settings property.
- * 
+ *
  * @param {*} config Must contain the _id property
  * @returns A copy of the config with secure properties appended (if any exist)
  */

diff --git a/service/src/transformers/authenticationconfiguration.js b/service/src/transformers/authenticationconfiguration.js
@@ -1,5 +1,6 @@
 "use strict";
 
+// TODO: users-next
 const AuthenticationConfiguration = require('../models/authenticationconfiguration');
 
 function transformAuthenticationConfiguration(authenticationConfiguration, options) {

diff --git a/service/src/utilities/authenticationApiAppender.js b/service/src/utilities/authenticationApiAppender.js
@@ -1,15 +1,16 @@
 "use strict";
 
 const extend = require('util')._extend
+    // TODO: users-next
     , AuthenticationConfiguration = require('../models/authenticationconfiguration')
     , AuthenticationConfigurationTransformer = require('../transformers/authenticationconfiguration');
 
 /**
- * Appends authenticationStrategies to the config.api object (the original config.api is not modified; this method returns a copy).  
+ * Appends authenticationStrategies to the config.api object (the original config.api is not modified; this method returns a copy).
  * These strategies are read from the db.
- * 
- * @param {*} api 
- * @param {*} options 
+ *
+ * @param {*} api
+ * @param {*} options
  * @returns Promise containing the modified copy of the api parameter
  */
 async function append(api, options) {