From 3929b3ca0a8f2b43d3ab05847e037aa2dfd4cf90 Mon Sep 17 00:00:00 2001
From: Girish Ramakrishnan <girish@cloudron.io>
Date: Tue, 28 Jan 2020 13:33:43 -0800
Subject: [PATCH] service: add memorySwap to configure route

---
 src/addons.js          | 1 -
 src/routes/services.js | 3 ++-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/addons.js b/src/addons.js
index 72f971a24..d8b0769cc 100644
--- a/src/addons.js
+++ b/src/addons.js
@@ -624,7 +624,6 @@ function updateServiceConfig(platformConfig, callback) {
 
     debug('updateServiceConfig: %j', platformConfig);
 
-    // TODO: this should possibly also rollback memory to default
     async.eachSeries([ 'mysql', 'postgresql', 'mail', 'mongodb', 'graphite' ], function iterator(serviceName, iteratorCallback) {
         const containerConfig = platformConfig[serviceName];
         let memory, memorySwap;
diff --git a/src/routes/services.js b/src/routes/services.js
index 297227100..ae0ce5607 100644
--- a/src/routes/services.js
+++ b/src/routes/services.js
@@ -40,10 +40,11 @@ function configure(req, res, next) {
     assert.strictEqual(typeof req.params.service, 'string');
 
     if (typeof req.body.memory !== 'number') return next(new HttpError(400, 'memory must be a number'));
+    if (typeof req.body.memorySwap !== 'number') return next(new HttpError(400, 'memorySwap must be a number'));
 
     const data = {
         memory: req.body.memory,
-        memorySwap: req.body.memory * 2
+        memorySwap: req.body.memorySwap
     };
 
     addons.configureService(req.params.service, data, function (error) {