style changes

Author: sedenardi

index.js

@@ -6,12 +6,12 @@ const authentication = require('./authentication');
 
 module.exports = {
   get: (event, context) => {
-    const sessionRes = session.get(event.headers);
-    if (sessionRes.valid) {
+    const sess = session.getSession(event.headers);
+    if (sess.valid) {
       render.user({
-        username: sessionRes.user.username,
-        first: sessionRes.user.first,
-        last: sessionRes.user.last
+        username: sess.user.username,
+        first: sess.user.first,
+        last: sess.user.last
       }, (err, res) => {
         if (err) { return context.done(err); }
         return context.done(null, res);
@@ -28,7 +28,7 @@ module.exports = {
     const pass = event.data.password;
     const authRes = authentication.auth(username, pass);
     if (authRes.success) {
-      const sess = session.set(authRes.user);
+      const sess = session.setSession(authRes.user);
       return context.done(null, {
         success: authRes.success,
         Cookie: sess.Cookie
@@ -38,8 +38,8 @@ module.exports = {
     }
   },
   logout: (event, context) => {
-    const sessionRes = session.get(event.headers);
-    const sess = session.destroy(sessionRes.user);
+    const sessionRes = session.getSession(event.headers);
+    const sess = session.destroySession(sessionRes.user);
     context.done(null, { Cookie: sess.Cookie });
   }
 };

render.js

@@ -8,13 +8,13 @@ const loginPath = path.resolve(__dirname, './login.html');
 const userPath = path.resolve(__dirname, './user.html');
 
 module.exports = {
-  login: function(cb) {
+  login: (cb) => {
     fs.readFile(loginPath, (err, res) => {
       if (err) { return cb(err); }
       return cb(null, res.toString());
     });
   },
-  user: function(opts, cb) {
+  user: (opts, cb) => {
     fs.readFile(userPath, (err, res) => {
       if (err) { return cb(err); }
       const compiled = template(res.toString());

session.js

@@ -7,7 +7,7 @@ const cookieKey = 'SID';
 const cookiePrefix = 'Session::';
 
 module.exports = {
-  get: (headers) => {
+  getSession: (headers) => {
     const cookieStr = headers ? (headers.Cookie || '') : '';
     const cookies = cookie.parse(cookieStr);
     if (!cookies[cookieKey]) {
@@ -20,16 +20,13 @@ module.exports = {
       user: user
     };
   },
-  set: (user) => {
+  setSession: (user) => {
     const sessionId = `${cookiePrefix}${user.username}`;
-    const newCookie = `${cookieKey}=${sessionId}`;
+    const newCookie = cookie.serialize(cookieKey, sessionId);
     return { Cookie: newCookie };
   },
-  destroy: (user) => {
-    // invalidate sessionId if user exists
-    const clearCookie = cookie.serialize(cookieKey, 'empty', {
-      maxAge: 0
-    })
+  destroySession: (user) => {
+    const clearCookie = cookie.serialize(cookieKey, 'empty', { maxAge: 0 });
     return { Cookie: clearCookie };
   }
 };