This is an automated email from the ASF dual-hosted git repository.

jpeach pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  ecfe141   TS-4383: Allow the trafficserver group to read from the 
log pipe.
ecfe141 is described below

commit ecfe1417836c9192dbdfb23efa31ec809a3f1d79
Author: David Calavera <[email protected]>
AuthorDate: Thu Apr 21 15:24:36 2016 -0700

    TS-4383: Allow the trafficserver group to read from the log pipe.
    
    ASCII_PIPE logs are created with 0600 privileges.  The owner of
    this pipe is `nobody` and the group is `nogroup`.  This is very
    limiting and it makes impossible to use the pipe to read logs.
    
    This closes #595.
    
    Signed-off-by: David Calavera <[email protected]>
---
 proxy/logging/LogFile.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/proxy/logging/LogFile.cc b/proxy/logging/LogFile.cc
index e700d21..2fcdfe1 100644
--- a/proxy/logging/LogFile.cc
+++ b/proxy/logging/LogFile.cc
@@ -159,7 +159,7 @@ LogFile::open_file()
 
   if (m_file_format == LOG_FILE_PIPE) {
     // setup pipe
-    if (mkfifo(m_name, S_IRUSR | S_IWUSR) < 0) {
+    if (mkfifo(m_name, S_IRUSR | S_IWUSR | S_IRGRP) < 0) {
       if (errno != EEXIST) {
         Error("Could not create named pipe %s for logging: %s", m_name, 
strerror(errno));
         return LOG_FILE_COULD_NOT_CREATE_PIPE;

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to