diff --git a/docs/courier-to-mailman3.md b/docs/courier-to-mailman3.md index 91ac0bd..13959a6 100644 --- a/docs/courier-to-mailman3.md +++ b/docs/courier-to-mailman3.md @@ -27,4 +27,5 @@ This is our current version trying to communicate between courier and mailman3. * Level and File set in variable * Format had errors with `$` used instead of `%` * Clean up logging to not log what was logged for logging diagnostics. -* Logging host and port on same line. \ No newline at end of file +* Logging host and port on same line. +* OK, parsing port number, and setting default. \ No newline at end of file diff --git a/source/courier-to-mailman3.py b/source/courier-to-mailman3.py index 19d1946..5ec8a94 100644 --- a/source/courier-to-mailman3.py +++ b/source/courier-to-mailman3.py @@ -31,11 +31,12 @@ try: # Setup LTMP connection lmtp_host = sys.argv[3] if len(sys.argv) > 3 else 'localhost' - logger.info('lmtp_host: %s lmtp_port: %s', lmtp_host, sysv.arg[1]) + lmtp_port = sys.argv[1] if len(sys.argv) > 1 else 8024 + logger.info('lmtp_host: %s lmtp_port: %s', lmtp_host, lmtp_port) # class smtplib.LMTP(host='', port=LMTP_PORT, local_hostname=None, source_address=None) - logger.info('lmtp_port: %s', sysv.arg[1]) - lmtp = smtplib.LMTP(lmtp_host, int(sys.argv[1])) - logger.info('lmtp_port (lmtp setup): %s', sysv.arg[1]) + logger.info('lmtp_port: %s', lmtp_port) + lmtp = smtplib.LMTP(lmtp_host, int(lmtp_port)) + logger.info('lmtp_port (lmtp setup): %s', lmpt_port) try: # Unfortunately qmail sends to local bare LF for end line, while