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
Copy Markdown

@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant