nixos/conduwuit: init

[niklaskorz]

As conduwuit deviates more and more from its conduit origins, it makes sense to give conduwuit its own module.
A custom module has been wished by upstream maintainers (@girlbossceo) as well as fellow nixpkgs maintainers (@nyabinary).

This allows us to cover breaking changes such as:

• conduwuit does not support sqlite as database backend (unlike conduit)
• conduwuit supports listening to unix sockets (unlike conduit)

Also includes the conduwuit-specific(?) fix from #353634.

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[nyabinary]

girlbossceo/conduwuit@08365bf

The option database_backend has been removed.

[nyabinary]

Maybe a release note should be added as well? Other than that it looks good to undraft?

[niklaskorz]

Maybe a release note should be added as well? Other than that it looks good to undraft?

I'm still trying to get the nixosTests to pass, currently getting:

conduwuit # [   14.639198] nginx[842]: 2024/11/17 19:18:45 [error] 842#842: *2 connect() failed (111: Connection refused) while connecting to upstream, client: 2001:db8:1::1, server: conduwuit, request: "POST /_matrix/client/v3/register HTTP/1.1", upstream: "http://[::1]:6167/_matrix/client/v3/register", host: "conduwuit"
client # Error validating response: 'user_id' is a required property

Edit: The "connection refused" simply came from the tests running before conduwuit was listening. Still getting this error though:

client # Error validating response: 'flows' is a required property
client # Error validating response: 'user_id' is a required property
client # Traceback (most recent call last):
client #   File "/run/current-system/sw/bin/do_test", line 51, in <module>
client #     asyncio.get_event_loop().run_until_complete(main())
client #   File "/nix/store/px2nj16i5gc3d4mnw5l1nclfdxhry61p-python3-3.12.7/lib/python3.12/asyncio/base_events.py", line 687, in run_until_complete
client #     return future.result()
client #            ^^^^^^^^^^^^^^^
client #   File "/run/current-system/sw/bin/do_test", line 22, in main
client #     response = await client.room_create(federate=False)
client #                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
client #   File "/nix/store/gaqh8hck8134gf95ikccqbx4zmp313p9-python3-3.12.7-env/lib/python3.12/site-packages/nio/client/base_client.py", line 115, in wrapper
client #     self._assert_logged_in()
client #   File "/nix/store/gaqh8hck8134gf95ikccqbx4zmp313p9-python3-3.12.7-env/lib/python3.12/site-packages/nio/client/base_client.py", line 357, in _assert_logged_in
client #     raise LocalProtocolError("Not logged in.")
client # nio.exceptions.LocalProtocolError: Not logged in.

[niklaskorz]

Tests are passing now 🎊

[cafkafk]

Great work, thanks for the effort :3

[niklaskorz]

Done addressing all feedback, fixing the tests right now, then we're good to go.

Edit: Tests are ready as well.

[niklaskorz]

Resolved merge conflicts in release notes

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/2147