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.
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
Dante improvements #39005
Dante improvements #39005
Conversation
Add PAM, SASL and UPnP support.
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: dante Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: dante Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: dante Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: dante Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: dante Partial log (click to expand)
|
@abbradar why remove |
Also, can we run with |
@Shados Ah, so it really can abruptly stop? I should have asked before making the change -- can you give an example so that we put it into comments and someone won't try to change it again? @peterhoeg Dante has some parts which run as privileged user, namely PAM authentication module. |
@abbradar Sorry, I don't remember the specific issues I ran into, and I can't check my non-existent logs; because I'm using it as part of a micro-VPN service I run for a few mates, I have logging for it disabled unless I'm actively looking into some problem. |
@Shados Got that; I'll just make a generic comment then. |
@Shados Given that you have disabled logs for it FWIW my instance doesn't log anything apart from "started up" and messages from PAM about successful authorizations. |
@abbradar No, having |
There is also |
Pushed an update to only restart on failure with a comment. |
Normal exit code shouldn't result in a restart.
Log to journald via syslog by default; also improve option type.
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: dante Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: dante Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: dante Partial log (click to expand)
|
Merging this in several days if no problems are found. |
Motivation for this change
Continuation on https://www.theverge.com/2018/4/13/17233112/russia-telegram-ban-court-ruling
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)cc @Shados