Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add positional information to all parse error messages. #817

Open
squell opened this issue Jan 15, 2024 · 0 comments
Open

Add positional information to all parse error messages. #817

squell opened this issue Jan 15, 2024 · 0 comments
Labels
C-checker Permission checking logic C-parser Parser/AST enhancement New feature or request

Comments

@squell
Copy link
Member

squell commented Jan 15, 2024

Some error messages in the sudoers file (e.g. an include file not found, recursive aliases) give correct parse error message but don't reveal the location. We could add this by adding location info in the AST for selected AST types. (Related to issue #806 and PR #815)

Possible exception would be recursive alias definitions (those don't really originate from a single location, and simply identifying the problematic alias gives enough clues on how to fix them in a sudoers config).

@squell squell added enhancement New feature or request C-parser Parser/AST C-checker Permission checking logic labels Aug 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-checker Permission checking logic C-parser Parser/AST enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant