chore: enable masking emails for invites

Author: maxceem

src/util.js

@@ -16,7 +16,7 @@ import config from 'config';
 import urlencode from 'urlencode';
 import elasticsearch from 'elasticsearch';
 import AWS from 'aws-sdk';
-// import jp from 'jsonpath';
+import jp from 'jsonpath';
 import Promise from 'bluebird';
 import models from './models';
 
@@ -617,16 +617,7 @@ _.assignIn(util, {
    *
    * @return {Object} data has been processed
    */
-  maskInviteEmails: (jsonPath, data, req) => { // eslint-disable-line
-    // temporary disable this feature, because it has some side effects
-    // see relative issues:
-    // - https://github.com/topcoder-platform/tc-project-service/issues/420
-    // - https://github.com/appirio-tech/connect-app/issues/3412
-    // - https://github.com/topcoder-platform/tc-project-service/issues/422
-    // - https://github.com/appirio-tech/connect-app/issues/3413
-    // uncomment code below, to enable masking emails again
-
-    /*
+  maskInviteEmails: (jsonPath, data, req) => {
     const isAdmin = util.hasPermission({ topcoderRoles: ADMIN_ROLES }, req.authUser);
     if (isAdmin) {
       return data;
@@ -639,7 +630,6 @@ _.assignIn(util, {
       // isString or null
       return util.maskEmail(value);
     });
-    */
     return data;
   },
 

src/util.spec.js

@@ -43,7 +43,7 @@ describe('Util method', () => {
     });
   });
 
-  xdescribe('maskInviteEmails', () => {
+  describe('maskInviteEmails', () => {
     it('should mask emails when passing data like for a project list endpoint for non-admin user', () => {
       const list = [
         {
@@ -70,6 +70,7 @@ describe('Util method', () => {
       };
       util.maskInviteEmails('$..invites[?(@.email)]', list, res).should.deep.equal(list2);
     });
+
     it('should mask emails when passing data like for a project details endpoint for non-admin user', () => {
       const detail = {
         id: 1,