return db settings only to client, sanitize keys

Author: ajhollid

server/controllers/settingsController.js

@@ -11,16 +11,15 @@ class SettingsController {
 	}
 
 	getAppSettings = async (req, res, next) => {
-		try {
-			const settings = { ...(await this.settingsService.getSettings()) };
-			delete settings.jwtSecret;
-			return res.success({
-				msg: this.stringService.getAppSettings,
-				data: settings,
-			});
-		} catch (error) {
-			next(handleError(error, SERVICE_NAME, "getAppSettings"));
+		const dbSettings = await this.settingsService.getDBSettings();
+		const sanitizedSettings = { ...dbSettings };
+		if (typeof sanitizedSettings.pagespeedApiKey !== "undefined") {
+			sanitizedSettings.pagespeedApiKey = "********";
 		}
+		return res.success({
+			msg: this.stringService.getAppSettings,
+			data: sanitizedSettings,
+		});
 	};
 
 	updateAppSettings = async (req, res, next) => {
@@ -32,7 +31,6 @@ class SettingsController {
 		}
 
 		try {
-			console.log(req.body);
 			await this.db.updateAppSettings(req.body);
 			const updatedSettings = { ...(await this.settingsService.reloadSettings()) };
 			delete updatedSettings.jwtSecret;

server/service/settingsService.js

@@ -54,6 +54,11 @@ class SettingsService {
 		}
 		return this.settings;
 	}
+
+	async getDBSettings() {
+		const settings = await this.appSettings.findOne({ singleton: true }).lean();
+		return settings;
+	}
 }
 
 export default SettingsService;