Use primary group of specified user
Brought to you by:
jmaggard
Menu
▾
▴
1 Attachments
#171 Use primary group of specified user
Commit 7a27ce introduced a command line option to set group. Is it necessary?
What do you think about the attached patch? (reverting the above commit first)
I think it matches better what users would expect when they set an user: it will also change to the primary group of that user.
I am not the project owner and cannot say if it is necessary or not. Having different user/group names can be helpful with more complex file sharing needs. Especially since minidlna is only able to limit which files are served by their permissions.
Sure. Traditionally this is done changing the user's primary group.
Separate setgid() error handling. No functional change.
Commit ad5d64 added this feature. But files (pidfile dir, pidfile, db_path) created before setuid/setgid still need ownership change to the new gid. Attached patch does that.
last version is here https://sourceforge.net/u/marcosfrm/minidlna/ci/734abac7a9b7a83d4ff88563e186ca9ab49c004e/