Skip to content

Add postfix conf script port 25 check#359

Merged
JedMeister merged 2 commits into
turnkeylinux:19.x-devfrom
JedMeister:postfix
Jun 3, 2026
Merged

Add postfix conf script port 25 check#359
JedMeister merged 2 commits into
turnkeylinux:19.x-devfrom
JedMeister:postfix

Conversation

@JedMeister
Copy link
Copy Markdown
Member

@JedMeister JedMeister commented Jun 1, 2026

Adds a check to ensure that port 25 is free. Previously it would silently error and postfix had particularly unhelpful error message so it was a PITA to debug!

FWIW I had to fab-investigate my root.patched, install rsyslog, start it running, rerun postfix and read the resulting log. One of the downsides of systemd journal...

@JedMeister JedMeister requested a review from OnGle June 1, 2026 06:02
Copy link
Copy Markdown
Member

@OnGle OnGle left a comment

Choose a reason for hiding this comment

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

lgtm!

@JedMeister
Copy link
Copy Markdown
Member Author

Also as noted by @OnGle offline - I've fixed the HOSTNAME test - as per the commit message the ! worked, but using -z is "more correct" and clearer.

@JedMeister JedMeister merged commit aaaef3b into turnkeylinux:19.x-dev Jun 3, 2026
@JedMeister JedMeister deleted the postfix branch June 3, 2026 05:10
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.

3 participants