Drift/pvv-nixos-config
17
5
Fork 1
Code Issues Pull Requests 7 Actions Wiki Activity

Roundcube testing on bekkalokk now working. #14

Merged
oysteikt merged 2 commits from roundcube into main 2023-11-26 05:17:29 +01:00
Conversation 4 Commits 2 Files Changed 3 +90
adriangl commented 2023-10-29 02:51:37 +01:00
Owner
Copy Link

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

Should we add roundcube to bekkalokk to have one less dependency on spikkjeposhe.
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
Author
Owner
Copy Link

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 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
Author
Owner
Copy Link

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
Owner
Copy Link

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
Owner
Copy Link

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

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
Owner
Copy Link

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
Owner
Copy Link

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

We have SSL smtp which we should probably use over STARTTLS on 25
adriangl commented 2023-11-02 11:33:30 +01:00
Author
Owner
Copy Link

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

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
Owner
Copy Link

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

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
Owner
Copy Link

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
Owner
Copy Link

We are no longer using starttls

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;
danio commented 2023-11-02 17:09:15 +01:00
Owner
Copy Link

nit: comments should be dropped

nit: comments should be dropped
adriangl force-pushed roundcube from 8400cecf64 to c8d383c9ab 2023-11-25 21:23:44 +01:00 Compare
oysteikt added 1 commit 2023-11-26 05:05:32 +01:00
bekkalokk: fix roundcube, and move to webmail2.pvv.ntnu.no/roundcube
All checks were successful
Eval nix flake / evals (push) Successful in 16m1s
Details
Eval nix flake / evals (pull_request) Successful in 21m4s
Details
2a1e649eed
oysteikt merged commit 54a54ad0f5 into main 2023-11-26 05:17:29 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
danio
Labels
Clear labels
art
The issue needs someone to be creative
 backup
big
No, not bug, **big** - this is gonna take a while
 blocked
This issue/PR depends on one or more other issues/PRs
 bug
Something is not working, and it's not the shield hero
 crash report
Report an oopsie
 disputed
kranglefanter
 documentation
Documentation changes required
 duplicate
This issue or pull request already exists
 enhancement
New feature
 good first issue
Get your hands dirty with a new project here
 logging
networking
TTM4100
 nixos
Requires changes to our nixos setup
 question
More information is needed
 salt
Requires changes to our salt setup
 security
Skommel
 servers n' hardware
Regards hardware, servers or VMs
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
adriangl albertba alfhj amalieem bjornoka chrisfjo chrivi danio davidk dungby eirikwit felixalb frero jonmro jovre karolds knuta krisleri larshhan oysteikt pederbs root torsteno yorinad
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Drift/pvv-nixos-config#14
Delete Branch "roundcube"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?