Skip to content

Commit bc1017f

Browse files
committed
Fix: Eliminate duplicate max_nesting_level in xdebug.ini
Move max_nesting_level default (512) from hardcoded value in generateXdebugIni to the config defaults in normalizeXdebugConfig, allowing clean user overrides without duplicate ini entries.
1 parent 18cef14 commit bc1017f

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

builders/php.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,9 @@ const normalizeXdebugConfig = xdebug => {
7878
client_port: 9003,
7979
log: '/tmp/xdebug.log',
8080
idekey: '',
81-
config: {},
81+
config: {
82+
max_nesting_level: 512,
83+
},
8284
};
8385

8486
if (xdebug === true) return _.merge({}, defaults, {mode: 'debug'});
@@ -104,7 +106,6 @@ const generateXdebugIni = config => {
104106
const ini = [
105107
'; Generated by Lando PHP plugin',
106108
`xdebug.mode = ${config.mode}`,
107-
`xdebug.max_nesting_level = 512`,
108109
`xdebug.start_with_request = ${config.start_with_request}`,
109110
`xdebug.client_host = ${config.client_host}`,
110111
`xdebug.client_port = ${config.client_port}`,

0 commit comments

Comments
 (0)