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.

Sign up for GitHub

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

Jump to bottom

Management of the ftpasswd file via concat and as file resource fails #48

Closed
saz opened this issue Jun 12, 2024 · 2 comments · Fixed by #49
Closed

Management of the ftpasswd file via concat and as file resource fails #48

saz opened this issue Jun 12, 2024 · 2 comments · Fixed by #49
Assignees
@fraenki
Labels
bug

Comments

@saz
Copy link
Contributor

saz commented Jun 12, 2024

I've been running into an issue with a missing ftpd.passwd file. After some research, I stumbled upon puppetlabs/puppetlabs-concat#806 which seems to be the issue here.

I'll send a PR, which will use a file resource only if $manage_ftpasswd_file is set to false.
saz added a commit to saz/puppet-proftpd that referenced this issue Jun 12, 2024
@saz
don't use file resource, when concat is in use
dafcd97 
Fixes markt-de#48
@saz saz mentioned this issue Jun 12, 2024
Merged
@saz
Copy link
Contributor Author

saz commented Jun 12, 2024

@fraenki I hope it's fine to also simplify the concat part a bit within this PR. If not, please let me know and I'll split it

@fraenki fraenki self-assigned this Jun 17, 2024
@fraenki fraenki added the bug label Jun 17, 2024
@fraenki
Copy link
Member

fraenki commented Jun 17, 2024

Thanks for taking care of this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fraenki fraenki

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

don't use file resource, when concat is in use saz/puppet-proftpd
2 participants
@saz @fraenki