diff --git a/lib/inspect/checks/reachable.js b/lib/inspect/checks/reachable.js index 5692481..8960449 100644 --- a/lib/inspect/checks/reachable.js +++ b/lib/inspect/checks/reachable.js @@ -35,7 +35,8 @@ class reachableCheck extends Check { resource: path, status: true, expected: fileExists.expected, - actual: permission ? fileExists.actual : undefined + actual: permission ? fileExists.actual : undefined, + msg: fileExists.msg }); } } else { @@ -43,7 +44,8 @@ class reachableCheck extends Check { resource: path, status: false, expected: fileExists.expected, - actual: permission ? fileExists.actual : undefined + actual: permission ? fileExists.actual : undefined, + msg: fileExists.msg }); } } @@ -54,7 +56,8 @@ class reachableCheck extends Check { results.forEach((result) => { let message = chalk`{gray [${result.resource}]} {white status:} {gray ${result.status}} ` + (result.expected ? chalk`{white expected permission:} {gray ${result.expected}} ` : '') + - (result.actual ? chalk`{white actual permission:} {gray ${result.actual}}` : ''); + (result.actual ? chalk`{white actual permission:} {gray ${result.actual}} ` : '') + + (result.msg ? chalk`{white msg:} {gray ${result.msg}}` : ''); this.reporter.report(message, result.status === true); }); @@ -79,6 +82,7 @@ class reachableCheck extends Check { let status = fileExists; let actual = ''; let expected = '' + let msg; if (fileExists) { @@ -112,12 +116,16 @@ class reachableCheck extends Check { let usersGroups = (await this.connector.exec(`groups ${user}`)).stdout.split(':')[1].trim().split(' '); // if user is in the group of file/dir - if (usersGroups.includes(actualGroup)) + if (usersGroups.includes(actualGroup)){ status = status && actualGroupPermission.includes(p); + msg = `${user} is in ${actualGroup} group`; + } // if user is NOT in the group of file/dir - else + else{ status = status && actualOtherPermission.includes(p); + if(!status) msg = `${user} is NOT in ${actualGroup} group`; + } } } @@ -154,7 +162,8 @@ class reachableCheck extends Check { return { status, expected: expected, - actual: actual + (isWin ? ', user is currently not supported on Windows':'') + actual: actual + (isWin ? ', user is currently not supported on Windows':''), + msg } }