-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
xandikos: backport systemd socket activation #161690
Conversation
I'm a little bit baffled about your PR description and the actual changes. |
Yeah, you're right. Xandikos was always able to do that, this PR just adds that to the nginx configuration. |
Topic and changes fit together. But I mean the mismatching description:
|
Yeah, I think you are right, when I made this I still had xandikos-0.2.6 and was about to create a pull request for that too but noticed this and probably mixed up the pull requests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, could you also expand nixos/tests/xandikos.nix
to include a subtest for sockets usecase?
While writing a test I noticed that the permission of the socket are too strict to be useful. Changing the services UMask does not seem appropriate. We should probably put this off until systemd's socket activation makes it into xandikos, see jelmer/xandikos#155. |
38f15b8
to
bce1a19
Compare
This way systemd takes care of the owner and the permissions of the socket.
Rebased to fix merge conflict. |
Motivation for this change
Backported jelmer/xandikos#155 to add support for systemd socket activation. This also enables UNIX sockets with proper owner/permissions, because systemd creates them for us.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes