Skip to content
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

nixos/freeradius : init - Added freeradius service #34330

Closed
wants to merge 0 commits into from

Conversation

netixx
Copy link
Contributor

@netixx netixx commented Jan 27, 2018

Inspired from the dhcpd service implementation
Only 2 configurations options at the moment:
enabled and path to configfile

folder networking has been chosen because radius is mainly used for network device and network authentication (e.g. WPA2 Entreprise)

Motivation for this change

Freeradius package is already supported by nix, but no service exists for nixos. This patch adds basic support to start the freeradius service.

Things done

I tested the service configuration in my personal nix installation successfully.

  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@bjornfor
Copy link
Contributor

bjornfor commented Feb 4, 2018

Closed by mistake? I see this PR was made from your master branch. It's a good idea to work on non-master branches for PRs.

@netixx
Copy link
Contributor Author

netixx commented Feb 4, 2018

Yes, I opened #34587 with the correct branch :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants