Skip to content

nginx parser fails on commented lines#1694

Open
jwes wants to merge 1 commit intoCISOfy:masterfrom
jwes:work/improve_nginx_parsing
Open

nginx parser fails on commented lines#1694
jwes wants to merge 1 commit intoCISOfy:masterfrom
jwes:work/improve_nginx_parsing

Conversation

@jwes
Copy link

@jwes jwes commented Dec 5, 2025

on lines like

ssl_prefer_server_ciphers on; # Force server cipher order. 

the result of the parser would keep the ';', returning 'on;' which in turn would lead to a mismatch of

"on" != "on;"

This was caused by an incorrect order of the sed statements. Now first the comment is removed and afterwards the trailing ; gets removed.

on lines like	ssl_prefer_server_ciphers on; # Force server cipher order. the result of the parser would keep the ';', returning 'on;' which in turn would lead to a mismatch of "on" != "on;" This was caused by an incorrect order of the sed statements. Now first the comment is removed and afterwards the trailing ; gets removed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant