Menu

#18 Does not honor "allow other host" check box.

closed
nobody
None
5
2004-11-21
2004-10-29
jpa
No

When the check box is unchecked, it should not listen
for other hosts.
Under Windows, using the cmd netstat -ano
will show that MrPostman (pid 2240 in this case) is
listening on all interfaces instead of just 127.0.0.1
(localhost):

TCP 0.0.0.0:11110 0.0.0.0:0
LISTENING 2240
TCP 0.0.0.0:25250 0.0.0.0:0
LISTENING 2240

I would sugest the following (or similar) change to
PopServer() _AND_ SmtpServer():

Replace "listenSocket = new ServerSocket(port);"
with:

if(mpn.getUserprefs().getBoolean("AllowOtherHosts",
false))
{
// Allow anybody to connect, so listen on any/all
local addresses
listenSocket = new ServerSocket(port);
}
else
{
// Nobody from the outside is allowed to connect.
Only listen on localhost.
listenSocket = new ServerSocket(port, 50 /*default
value*/, InetAddress.GetLocalHost());
}

--
jpa

Discussion

  • jpa

    jpa - 2004-10-29
    • summary: Does not honour "allow other host" check box. --> Does not honor "allow other host" check box.
     
  • jpa

    jpa - 2004-10-29

    Logged In: YES
    user_id=1009183

    Change also MrPostmanServer.run(){}
    "listenSocket = new ServerSocket(port);"
    so that it listens mostly locally.

     
  • Martin Vlcek

    Martin Vlcek - 2004-10-31
    • status: open --> open-postponed
     
  • Martin Vlcek

    Martin Vlcek - 2004-10-31

    Logged In: YES
    user_id=1020004

    MrPostman will check the address of the client trying to
    connect. Thus I currently see no necessity for the change.
    Moved to Feature Requests.

     
  • jpa

    jpa - 2004-11-08
    • status: open-postponed --> open
     
  • jpa

    jpa - 2004-11-08

    Logged In: YES
    user_id=1009183

    Attached picture of Windows complaining about MrPostman
    listening on non-loopback IP address.
    (I'm done with a patch for this, and will be posting it in
    the next couple of minutes).

     
  • jpa

    jpa - 2004-11-08

    Windows XP SP2 firewall complaining about MrPostman

     
  • Martin Vlcek

    Martin Vlcek - 2004-11-21

    Logged In: YES
    user_id=1020004

    Should work now in MrPostman 1.1 RC1. Patch was applied
    (slightly modified).

     
  • Martin Vlcek

    Martin Vlcek - 2004-11-21
    • status: open --> closed
     

Log in to post a comment.