albertito / chasquid

SMTP (email) server with a focus on simplicity, security, and ease of operation [mirror]
https://blitiri.com.ar/p/chasquid/
Other
868 stars 56 forks source link

Log to Stdout #10

Closed denysvitali closed 4 years ago

denysvitali commented 4 years ago

Currently it is not possible to use /dev/stdout if chasquid is running as an unprivileged user since it will try to set a bad file mode on the special file (0664).

This PR adds the <stdout> syntaxt to the config file to log to stdout

albertito commented 4 years ago

Thank you for the patch!

I ended up re-doing this in a slightly different way, and added support for stderr, as well as tests. You can find the result in commit d9d5655, which is in the next branch for now.

I think that covers your use case, so I'm going to close this, but please feel free to reopen or comment further if you have any more issues or suggestions!

PS: You're mentioned in the commit message, if you prefer to change the format of the attribution please let me know. Since it's on the next branch, the commit can be amended so it's not too late :)

denysvitali commented 4 years ago

Thanks @albertito , your solution is way better than mine and the attribution looks good!