Skip to content

Commit 56e7b1c

Browse files
author
Igor Hrcek
authored
Merge pull request #36 from igorhrcek/custom-files-fix
fix: Fix BlockAccessToCustomSensitiveFiles
2 parents 2c935ac + bdc4003 commit 56e7b1c

3 files changed

Lines changed: 17 additions & 5 deletions

File tree

src/SubCommands/AddSecurityHeaders.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ class AddSecurityHeaders extends SubCommand {
99
public string $removalMessage= 'Add Security Headers rule has been removed.';
1010

1111
public function getTemplateVars() : array {
12+
1213
$default_headers = [
1314
'Strict-Transport-Security' => '"max-age=63072000; includeSubDomains; preload"',
1415
'Referrer-Policy' => 'strict-origin-when-cross-origin',

src/SubCommands/BlockAccessToCustomSensitiveFiles.php

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,21 @@ public function getTemplateVars(): array {
2020
$files_array = [];
2121

2222
foreach ($files as $key => $value) {
23-
$file = (isset($this->commandArguments['server']) && $this->commandArguments['server'] === 'nginx') ?
24-
preg_quote($value) : $value;
25-
26-
$files_array[] = ['file' => $file];
23+
if ( preg_match( '/.+\/.+/', $value ) ) {
24+
$file_with_directory = $this->setRuleContent( false, 'block_access_to_sensitive_files_with_directories' );
25+
if ( isset( $this->commandArguments['server'] ) && $this->commandArguments['server'] === 'nginx' ) {
26+
$file = $value;
27+
} else {
28+
$file = preg_quote( ltrim( $value, '/' ) );
29+
}
30+
$files_array[] = [ $file => $file_with_directory ];
31+
} else {
32+
$file = (isset($this->commandArguments['server']) && $this->commandArguments['server'] === 'nginx') ?
33+
preg_quote($value) : $value;
34+
$files_array[] = ['file' => $file];
35+
}
2736
}
28-
37+
2938
return $files_array;
3039
}
3140

src/SubCommands/BlockAccessToSensitiveFiles.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ class BlockAccessToSensitiveFiles extends SubCommand {
88
public string $successMessage = 'Block Access to Sensitive Files rule has been deployed.';
99
public string $removalMessage= 'Block Access to Sensitive Files rule has been removed.';
1010

11+
1112
/**
1213
* @var string List of files that we are protecting by default
1314
*/
@@ -37,4 +38,5 @@ public function getTemplateVars() : array {
3738
}
3839
return [];
3940
}
41+
4042
}

0 commit comments

Comments
 (0)