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

(profile::ccs::home) handle EL8+ login.defs home dir perms #1199

Open
wants to merge 1 commit into
base: production
Choose a base branch
from

Conversation

glennmorris
Copy link
Contributor

To make new home directories world-readable by default in EL8+, it is necessary to modify HOME_MODE in /etc/login.defs.

@glennmorris glennmorris added enhancement New feature or request ccs Camera Control System labels Jun 4, 2024
@glennmorris glennmorris force-pushed the IT-5402/ccs-home-dirs branch from 8905307 to 9e0c4ba Compare June 4, 2024 18:39
@glennmorris glennmorris force-pushed the IT-5402/ccs-home-dirs branch from 9e0c4ba to f1e2c3e Compare June 4, 2024 18:39
@glennmorris glennmorris marked this pull request as ready for review June 4, 2024 18:43
@glennmorris glennmorris requested a review from jhoblitt June 4, 2024 18:43
file_line { 'Change default home permissions for EL8+':
path => '/etc/login.defs',
match => '^HOME_MODE\s',
line => 'HOME_MODE 0755',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this would probably apply to home dirs created for IPA users as well. What is trying to be accomplished?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having it apply to IPA users is the wanted behaviour.
The intent is to make the home dirs of normal users world-readable by default, since this is a collaborative enterprise and people need to share work.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to wait until Cristian returns from leave next week for further discussion.

Copy link
Member

@jhoblitt jhoblitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is on hold pending further discussion.

@glennmorris
Copy link
Contributor Author

This is on hold pending further discussion.

OK.

If you want to keep them private (by default) for some security purposes, we'll live with it, but otherwise historically the CCS team's position was that we are all in this together, and these systems are designed for shared work towards common goals, not private silos.

@glennmorris
Copy link
Contributor Author

PS. the existing code that makes them public on CentOS 7 hosts has been in place for 4 years.
The changes in this pull request are because the syntax for doing that changed in RHEL8.

@glennmorris
Copy link
Contributor Author

So, any thoughts/discussion about this?

@glennmorris
Copy link
Contributor Author

No "further discussion" of any kind has happened in six months. 🤷‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ccs Camera Control System enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants