sergejmueller/wpcheck

View on GitHub
lib/rules/sensitive-files.js

Summary

Maintainability
A
2 hrs
Test Coverage
 
/**
* wpcheck module sensitive-files.js
* Check WordPress/Apache/Dot files for their availability
*/
 
 
/**
* Required modules
*/
 
const request = require( 'request' ).defaults( { followRedirect: false } )
const fs = require( '../fs' )
const log = require( '../log' )
 
 
/**
* Initiator method
*
* @param {Object} data Data object with request values
* @return void
*/
 
Function `fire` has 57 lines of code (exceeds 25 allowed). Consider refactoring.
exports.fire = ( data ) => {
 
const { wpURL, siteURL, userAgent, silentMode } = data
 
const filterName = fs.fileName( __filename, '.js' )
 
const logObj = { silentMode, filterName }
 
const targets = [
{
'url': `${wpURL}/wp-config.php`,
'method': 'HEAD',
'pattern': 'DB_PASSWORD'
},
{
'url': `${wpURL}/wp-admin/maint/repair.php`,
'method': 'HEAD',
'pattern': 'repair.php'
},
{
'url': `${siteURL}/.htaccess`
},
{
'url': `${siteURL}/.htpasswd`
},
{
'url': `${siteURL}/.ssh`
},
{
'url': `${siteURL}/.npmrc`
},
{
'url': `${siteURL}/.gitconfig`
},
{
'url': `${siteURL}/config.json`
},
{
'url': `${wpURL}/wp-config-sample.php`
},
{
'url': `${wpURL}/wp-content/debug.log`
}
]
 
targets.forEach( ( { url, method = 'GET', pattern = null } ) => {
 
request( {
'url': url,
'method': method,
'headers': { 'User-Agent': userAgent }
}, ( error, response, body ) => {
 
if ( error || response.statusCode !== 200 ) {
return log.ok( `${url} is not public`, logObj )
}
 
if ( ! pattern ) {
return log.warn( `${url} is public`, logObj )
}
 
if ( ! body.includes( pattern ) ) {
return log.info( `${url} is public but safe`, logObj )
}
 
return log.warn( `${url} is public and not safe`, logObj )
 
} )
 
} )
 
}