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

Update mail server configuration and random TCP port #7

Merged
merged 41 commits into from
May 2, 2024
Merged

Conversation

stephdl
Copy link
Collaborator

@stephdl stephdl commented Apr 23, 2024

This pull request includes updates to the mail server configuration and TCP ports. It also adds a script to restart the piler service when mail settings are changed.

the biggest change is we could have as many archive we want on the same server, we reclaim to the core two ports (one for traefik, one for smtp connection)

refs: NethServer/dev#6895

it needs NethServer/ns8-mail#109 to set the bcc in ns8-mail

When we first configure the archive, we import the emails, during the time the service runs to import we display a banner. the service could be started again by the CLI as many you need it.

When the restoration occurs for the example mail1 module, we start to restore a new module lets say piler2 and configure it, but when we remove piler1 if we have no errors, the destroy-module action remove the always bcc set in the mail server and let unconfigured always_bcc for piler2. So I display a banner in UI to let the admin knows that the always_bcc is not set in the mail server (just press save to configure again the piler module)

Capture d’écran du 2024-04-27 08-00-02
Capture d’écran du 2024-04-27 08-01-00
Capture d’écran du 2024-04-27 08-02-03

Copy link
Member

@DavidePrincipi DavidePrincipi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to rename action set-bcc-piler :)

@stephdl stephdl force-pushed the sdl-6895 branch 3 times, most recently from 7254b49 to ae6cf75 Compare April 24, 2024 10:40
@stephdl stephdl force-pushed the sdl-6895 branch 2 times, most recently from 14d63b5 to c21d7dd Compare April 24, 2024 13:51
@stephdl stephdl force-pushed the sdl-6895 branch 3 times, most recently from 528cc8c to e52bb10 Compare April 24, 2024 14:44
Copy link
Member

@DavidePrincipi DavidePrincipi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but...

@DavidePrincipi DavidePrincipi mentioned this pull request Apr 24, 2024
19 tasks
stephdl added 2 commits April 26, 2024 14:20
…lay and set BCC address using os.environ['MODULE_ID']
@stephdl stephdl requested a review from DavidePrincipi April 27, 2024 06:33
@stephdl
Copy link
Collaborator Author

stephdl commented Apr 27, 2024

Should be good @DavidePrincipi

Copy link
Member

@DavidePrincipi DavidePrincipi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stephdl stephdl merged commit e129403 into main May 2, 2024
1 check passed
@stephdl stephdl deleted the sdl-6895 branch May 2, 2024 14:31
@stephdl stephdl self-assigned this May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants