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

[rule/allows_nss_options]: Attribute 'offline_timeout_random_offset' is not allowed in section 'nss'. Check for typos #7289

Open
georgemarselis-nvi opened this issue Apr 11, 2024 · 5 comments
Labels
man-page Man page related issues

Comments

@georgemarselis-nvi
Copy link

sssd.conf(5) mentions offline_timeout_random_offset to be part of [nss]. sssctl config-check says otherwise.

Am i reading this wrong?

@sumit-bose
Copy link
Contributor

Hi,

thank you for the report, looks like all the offline_timeout* options are in the wrong section. They should be in DOMAIN SECTIONS.

Would you like to create a pull-request moving those options?

I'll close the other two tickets as duplicates of this one.

bye,
Sumit

@georgemarselis-nvi
Copy link
Author

Hi,

thank you for the report, looks like all the offline_timeout* options are in the wrong section. They should be in DOMAIN SECTIONS.

Would you like to create a pull-request moving those options?

I'll close the other two tickets as duplicates of this one.

bye, Sumit

sure, i'll clone and submit something

@georgemarselis-nvi
Copy link
Author

georgemarselis-nvi commented Apr 11, 2024

DOMAIN SECTIONS

Do you have any suggestion where in DOMAIN SECTIONS i can insert the offline_timeout* parameters please? The parameters do not seem to be organized either in alphabetical order, nor in order of importance.

@sumit-bose
Copy link
Contributor

DOMAIN SECTIONS

Do you have any suggestion where in DOMAIN SECTIONS i can insert the offline_timeout* parameters please? The parameters do not seem to be organized either in alphabetical order, nor in order of importance.

Hi,

I would suggest to either put then after entry_cache_computer_timeout or at the end following auto_private_groups.

bye,
Sumit

georgemarselis-nvi added a commit to georgemarselis-nvi/sssd that referenced this issue Apr 11, 2024
moved offline_timeout* from section [NSS] to section [DOMAIN SECTIONS]

also: 

* added spaces between options to make it easier to the eye
* xml comment as section divider
@georgemarselis-nvi
Copy link
Author

#7290

@andreboscatto andreboscatto added the man-page Man page related issues label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
man-page Man page related issues
Projects
None yet
Development

No branches or pull requests

3 participants