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

Fails to parse include and includedir files #504

Open
johnnybubonic opened this issue Jan 4, 2023 · 1 comment
Open

Fails to parse include and includedir files #504

johnnybubonic opened this issue Jan 4, 2023 · 1 comment

Comments

@johnnybubonic
Copy link

Currently, the include and includedir directives of krb5.conf are not parsed. They are not even considered.
They are indeed a canonical directive of krb5.conf:

The krb5.conf file can include other files using either of the following directives at the beginning of a line:

include FILENAME
includedir DIRNAME

FILENAME or DIRNAME should be an absolute path. The named file or directory must exist and be readable. Including a directory includes all files within the directory whose names consist solely of alphanumeric characters, dashes, or underscores. Included profile files are syntactically independent of their parents, so each included file must begin with a section header.

Because of this, this library entirely fails to work properly on e.g. RHEL/EL Linux distributions which have been joined to a domain via sssd, as SSSD makes use of nested includedir directives to manage Kerberos.

I suspect you could simply do a "pre-pass" parse, searching only for lines that have an include or includedir directive, and replacing those lines in-place with the referenced file(s') contents.

@bendylan
Copy link

bendylan commented Aug 9, 2023

Just wanted to +1 on this issue, would be nice to have a fix to use this library on EL

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

No branches or pull requests

2 participants