Jeremy Evans <c...@jeremyevans.net> wrote: > This patch allows Worker#user to accept the group argument as an array > of two strings, the first string being the process primary group, and > the second string being the group that owns the log files. This can > help when you have a large number of applications that use unique > primary groups, and want to have a user with the ability to read the log > files for any of the applications, especially if you are on an operating > system that only supports a small number of groups per user.
Fwiw, I don't understand why there's a blurb here for an isolated patch when your commit message below is is sufficient :) Along the same lines, "--cover-letter" in git-format-patch(1) isn't necessary for single patches, either, but they help with a multi-patch series. > Subject: [PATCH] Make Worker#user support different process primary group and > log file group > > Previously, Unicorn always used the process's primary group as the > the group of the log file. However, there are reasons to use a > separate group for the log files, such as when you have many > applications where each application uses it's own user and primary > group, but you want to be able to have a user read the log files > for all applications. Some operating systems have a fairly small > limit on the number of groups per user, and it may not be feasible > to have a user be in the primary group for all applications. > a primary group Anyways, this featureseems acceptable. Pushed to master as 47fddb53aa0b7763f353ba515cf3fb5b2059f4f7 Thanks -- unsubscribe: unicorn-public+unsubscr...@bogomips.org archive: https://bogomips.org/unicorn-public/
