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.

Sign up for GitHub

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

Jump to bottom

Enable LMTP over TCP #123

Closed
wants to merge 4 commits into from
Closed

Enable LMTP over TCP #123

wants to merge 4 commits into from

Commits on Oct 17, 2020

  1. enable LMTP over TCP 

    remove check that forbids LMTP over TCP
introduce Network config option, to choose between TCP and unix sockets
add a warning to the documentation that LMTP should not be used for WAN communication
    @patrick246
    patrick246 committed Oct 17, 2020
    Configuration menu
    Copy the full SHA
    d4461f2 View commit details
    Browse the repository at this point in the history

  2. add more tests for lmtp over tcp 

    test default Network config behavior
test error for empty lmtp address
test lmtp listening on tcp port
    @patrick246
    patrick246 committed Oct 17, 2020
    Configuration menu
    Copy the full SHA
    e9d393d View commit details
    Browse the repository at this point in the history

Commits on Oct 20, 2020

  1. split listen part from ListenAndServe, to make the setup code better … 

    …testable
    @patrick246
    patrick246 committed Oct 20, 2020
    Configuration menu
    Copy the full SHA
    9a3e1fd View commit details
    Browse the repository at this point in the history

  2. change "no port defined" error message to "no address defined"

    @patrick246
    patrick246 committed Oct 20, 2020
    Configuration menu
    Copy the full SHA
    46f41b4 View commit details
    Browse the repository at this point in the history