Skip to content

nordvpn: init at 4.3.1, maintainers: add different-error #477174

Open
different-error wants to merge 2 commits intoNixOS:masterfrom
different-error:nordvpn-pkg-gui
Open

nordvpn: init at 4.3.1, maintainers: add different-error #477174
different-error wants to merge 2 commits intoNixOS:masterfrom
different-error:nordvpn-pkg-gui

Conversation

@different-error
Copy link

@different-error different-error commented Jan 5, 2026

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

This PR contains just the package found in #406725.

I manually verified vpn connection over OpenVPN and NordLynx using both cli and gui.

configuration.nix

{
  pkgs,
  ...
}:

{
  imports = [
    ./hardware-configuration.nix
  ];

  boot.loader.systemd-boot.enable = true;
  boot.loader.efi.canTouchEfiVariables = true;
  networking.firewall.enable = true;
  networking.firewall.checkReversePath = "loose";

  services.xserver.enable = true;
  services.displayManager.gdm.enable = true;
  services.desktopManager.plasma6.enable = true;

  virtualisation.vmVariant = {
    virtualisation = {
      memorySize = 8192;
      cores = 3;
    };
  };
  users.groups.alice = { };
  users.groups.nordvpn = { };
  users.users.alice = {
    isSystemUser = true;
    password = "alice";
    group = "alice";
    extraGroups = [
      "wheel"
      "nordvpn"
    ];
    shell = pkgs.bash;
    home = "/home/alice";
    createHome = true;
    packages = with pkgs; [
      tree
      tmux
      dunst
      libnotify
      socat
      vim
      nordvpn
    ];
  };
  system.stateVersion = "24.11"; # Did you read the comment?
}

To build, nixos-rebuild build-vm -I nixos-config=/path/to/configuration.nix -I nixpkgs=/path/to/nixpkgs --max-jobs 4

Inside vm:

sudo nordvpnd
sudo chown alice:alice /run/nordvpn/nordvpnd.sock
# login to nordvpn using nordvpn login or nordvpn login --token, not shown here
nordvpn set technology nordlynx && nordvpn c
nordvpn set technology openvpn && nordvpn c
nordvpn-gui

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.first-time contribution This PR is the author's first one; please be gentle! 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` labels Jan 5, 2026
@different-error different-error changed the title Nordvpn pkg gui nordvpn: init at 4.3.1, maintainers: add different-error Jan 5, 2026
@different-error
Copy link
Author

@ruffsl @andersonjoseph fyi.

I would appreciate your manual testing of this PR. Thank you.

@ruffsl
Copy link
Contributor

ruffsl commented Jan 10, 2026

I tried this along with the networking service module from #406725 :

  • compilation took a lot longer than before, not sure I incorporated it into my local config correctly.
    • perhaps it because I need this to nix compile locally? Probably wrong.
-          extraPreBuild + (old.preBuild or "");
+          extraPreBuild + (if old ? preBuild then old.preBuild else "");
  • had to sudo systemctl stop nordvpnd.service and wipe /var/lib/nordvpn/data to switch to this branch
  • CLI works fine for login and connecting to servers
  • GUI no longer installs a desktop icon
    • had to launch system tray manually via systemctl start --user norduserd.service
    • system tray menu otherwise works as expected
  • Desktop notifications for connect and disconnections working as expected
  • tested VPN protocols via GUI: NordLynx, OpenVPN (TCP) and (UDP) look to be working
    • but NordWhisper would not toggle: "Something went wrong. Please try again." when selected

@different-error
Copy link
Author

  * perhaps it because I need this to nix compile locally? Probably wrong.
-          extraPreBuild + (old.preBuild or "");
+         extraPreBuild + (if old ? preBuild then old.preBuild else "");

Thanks for pointing this out. I think the old.preBuild is standard and should exist. Regardless, this critique is out-of-scope of this PR which does not include the referenced module.


* had to `sudo systemctl stop nordvpnd.service` and wipe `/var/lib/nordvpn/data` to switch to this branch

I had to do this once before because of the salt removal from previous iterations of this package. New consumers needn't do this. And once again, the module is out of scope of this PR.


* GUI no longer installs a desktop icon

I've fixed this now, ty.


  * had to launch system tray manually via `systemctl start --user norduserd.service`

If you use the standalone package and call sudo nordvpnd in the terminal, then nordvpnd would've spawned norduserd processes on its own.


  * but NordWhisper would not toggle: "Something went wrong. Please try again." when selected

nordwhisper (specifically libquench) as I understand it is closed source.

@different-error different-error marked this pull request as ready for review January 14, 2026 11:25
@nixpkgs-ci nixpkgs-ci bot added the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Jan 14, 2026
@nixpkgs-ci nixpkgs-ci bot added the 8.has: package (new) This PR adds a new package label Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 8.has: package (new) This PR adds a new package 9.needs: reviewer This PR currently has no reviewers requested and needs attention. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.first-time contribution This PR is the author's first one; please be gentle!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants