Roundcube testing on bekkalokk now working. #14

adriangl · 2023-10-29T02:51:37+01:00

adriangl commented

2023-10-29 02:51:37 +01:00

Should we add roundcube to bekkalokk to have one less dependency on spikkjeposhe.

adriangl added 1 commit 2023-10-29 02:51:38 +01:00

bekkalokk: actiated roundcube test

Eval nix flake / evals (push) Successful in 2m48s

Details

Eval nix flake / evals (pull_request) Successful in 2m38s

Details

d4f0330018

adriangl force-pushed roundcube from d4f0330018 to c2bcd7d18f

2023-10-29 03:03:53 +01:00

Compare

adriangl force-pushed roundcube from c2bcd7d18f to 6139cb8c19

2023-10-29 03:13:37 +01:00

Compare

adriangl commented

2023-10-29 03:18:52 +01:00

Nevermind smtp is currently broken and needs to be fixed.

I currently get:

roundcube SMTP Error (451): Failed to add recipient

and have gotten:

SMTP Error (535): Authentication failed

before commenting out.


# $config['smtp_user'] = "%u@pvv.ntnu.no";
# $config['smtp_pass'] = "%p";

Nevermind smtp is currently broken and needs to be fixed. I currently get: roundcube SMTP Error (451): Failed to add recipient and have gotten: SMTP Error (535): Authentication failed before commenting out. ``` # $config['smtp_user'] = "%u@pvv.ntnu.no"; # $config['smtp_pass'] = "%p"; ```

adriangl added 1 commit 2023-10-29 03:27:18 +01:00

lol

Eval nix flake / evals (push) Successful in 4m10s

Details

Eval nix flake / evals (pull_request) Successful in 4m35s

Details

4f57c9531d

adriangl added 1 commit 2023-10-29 03:32:21 +01:00

lol2

Eval nix flake / evals (push) Successful in 5m33s

Details

Eval nix flake / evals (pull_request) Successful in 16m4s

Details

8ae5a64cc8

adriangl force-pushed roundcube from 8ae5a64cc8 to 7dc0b99a09

2023-10-29 03:35:01 +01:00

Compare

adriangl commented

2023-10-29 22:14:01 +01:00

Maybe fixed with changing user to pvv.ntnu.no instead of imap.
thanx @felixalb

$config['mail_domain'] = "pvv.ntnu.no";
$config['smtp_user'] = "%u";

Maybe fixed with changing user to pvv.ntnu.no instead of imap. thanx @felixalb $config['mail_domain'] = "pvv.ntnu.no"; $config['smtp_user'] = "%u";

danio requested changes 2023-10-30 09:46:07 +01:00

danio left a comment

Please fix the commit author and message

Maybe something like:

bekkalokk/roundcube: init at <listen url>

Please fix the commit author and message Maybe something like: bekkalokk/roundcube: init at `<listen url>`

hosts/bekkalokk/configuration.nix Outdated

						
				@ -13,6 +13,7 @@

				    # ./services/website.nix

				    ./services/nginx.nix

				    ./services/gitea/default.nix

				    ./services/webmail

danio commented

2023-10-30 09:36:09 +01:00

praise: Thanks for putting it in a folder like this!

danio marked this conversation as resolved

hosts/bekkalokk/services/webmail/roundcube.nix Outdated

						
				@ -0,0 +3,4 @@

				with lib;

				let

				  cfg = config.services.roundcube;

				  domain = "webmail2.pvv.ntnu.no";

danio commented

2023-10-30 09:45:58 +01:00

webmail is currently a directory of multiple email clients, but the roundcube module in nixos listens on / path on this host so this should probably be a service domain like roundcube.pvv.ntnu.no

We can set up a redirect on webmail.pvv.ntnu.no/roundcube later to keep old links/bookmarks working

Alternatively you could override the nginx config to listen on the subdirectory

`webmail` is currently a directory of multiple email clients, but the roundcube module in nixos listens on `/` path on this host so this should probably be a service domain like `roundcube.pvv.ntnu.no` We can set up a redirect on `webmail.pvv.ntnu.no/roundcube` later to keep old links/bookmarks working Alternatively you could override the nginx config to listen on the subdirectory

danio marked this conversation as resolved

hosts/bekkalokk/services/webmail/roundcube.nix Outdated

						
				@ -0,0 +22,4 @@

				        $config['default_host'] = "ssl://imap.pvv.ntnu.no";

				        $config['default_port'] = 993;

				        $config['smtp_server'] = "tls://smtp.pvv.ntnu.no";

				        $config['smtp_port'] = 25;

danio commented

2023-10-30 09:37:53 +01:00

We have SSL smtp which we should probably use over STARTTLS on 25

adriangl commented

2023-11-02 11:33:30 +01:00

Sure. I only copied our current roundcube config and worked a bit on it to work.

danio marked this conversation as resolved

hosts/bekkalokk/services/webmail/roundcube.nix Outdated

						
				@ -0,0 +32,4 @@

				      '';

				  };  

				  networking.firewall.allowedTCPPorts = [ 80 443 ];

danio commented

2023-10-30 09:40:06 +01:00

nginx is what listens on these ports, and that is managed somewhere else so this should be removed

danio marked this conversation as resolved

adriangl force-pushed roundcube from 7dc0b99a09 to 7aae4639d0

2023-11-02 11:38:42 +01:00

Compare

adriangl force-pushed roundcube from 7aae4639d0 to a7e9c52d9f

2023-11-02 11:42:57 +01:00

Compare

adriangl force-pushed roundcube from a7e9c52d9f to 8400cecf64

2023-11-02 11:44:01 +01:00

Compare

danio requested changes 2023-11-02 17:10:20 +01:00

danio left a comment

The commit author is still "=" instead of adrian, would be nice to get fixed

That is the only thing I consider super-blocking

The commit author is still "=" instead of adrian, would be nice to get fixed That is the only thing I consider super-blocking

hosts/bekkalokk/services/webmail/roundcube.nix Outdated

						
				@ -0,0 +16,4 @@

				      extraConfig = ''

				        # starttls needed for authentication, so the fqdn required to match

				        # the certificate

danio commented

2023-11-02 17:08:29 +01:00

We are no longer using starttls

hosts/bekkalokk/services/webmail/roundcube.nix Outdated

						
				@ -0,0 +21,4 @@

				        $config['default_host'] = "ssl://imap.pvv.ntnu.no";

				        $config['default_port'] = 993;

				        #$config['smtp_server'] = "tls://smtp.pvv.ntnu.no";

				        #$config['smtp_port'] = 25;