Skip to content

Commit bc618ba

Browse files
committed
Fix: Ensure non-configured rules default to empty array
If a developer declines to set a value for `moduleSettings.cbSecurity.rules`, they get an error: ``` Element SETTINGS.RULES is undefined in ARGUMENTS. ``` IMHO, this is a failure on Coldbox's part to properly merge the dev-defined coldbox `moduleSettings.cbSecurity` struct with cbSecurity's own `settings` struct. Regardless, I'd expect the `rules` array to default to an empty array _[as documented](https://coldbox-security.ortusbooks.com/getting-started/first-chapter)_.
1 parent a6dc311 commit bc618ba

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

models/util/RulesLoader.cfc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,7 @@ component accessors="true" singleton {
6363
*/
6464
function rulesSourceChecks( required settings ){
6565
param arguments.settings.rulesSource = "";
66+
param arguments.settings.rules = [];
6667

6768
// Auto detect rules source
6869
if ( isSimpleValue( arguments.settings.rules ) ) {

0 commit comments

Comments
 (0)