Skip to content
This repository has been archived by the owner on Mar 6, 2023. It is now read-only.

Do not use createhome alias as it seems to break whole system permissions #111

Merged
merged 1 commit into from
Oct 16, 2019
Merged

Do not use createhome alias as it seems to break whole system permissions #111

merged 1 commit into from
Oct 16, 2019

Conversation

paulfantom
Copy link
Member

@paulfantom paulfantom commented Oct 16, 2019 •
edited
Loading

When module user uses createhome alias instead of create_home it can break system permissions.

Alternative approach to #110
Closes #110

@paulfantom paulfantom added invalid This doesn't seem right wontfix This will not be worked on labels Oct 16, 2019
@stale stale bot removed the wontfix This will not be worked on label Oct 16, 2019
@paulfantom paulfantom changed the title Update install.yml Do not use createhome alias as it seems to break whole system permissions Oct 16, 2019
@paulfantom paulfantom changed the title Do not use createhome alias as it seems to break whole system permissions [WIP] Do not use createhome alias as it seems to break whole system permissions Oct 16, 2019
@paulfantom paulfantom added bug Something isn't working and removed invalid This doesn't seem right labels Oct 16, 2019
@paulfantom paulfantom mentioned this pull request Oct 16, 2019
Closed
SuperQ
SuperQ approved these changes Oct 16, 2019
View reviewed changes
Copy link
Collaborator

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

LGTM

@paulfantom paulfantom changed the title [WIP] Do not use createhome alias as it seems to break whole system permissions Do not use createhome alias as it seems to break whole system permissions Oct 16, 2019
@paulfantom paulfantom merged commit 2477fe8 into master Oct 16, 2019
@paulfantom paulfantom deleted the createhome branch October 16, 2019 22:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@SuperQ SuperQ SuperQ approved these changes

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@paulfantom @SuperQ