Better logging while establishing and shutting down connections.

Results of the resolver are now discarded after the client is registered.

diff --git a/ChangeLog b/ChangeLog
index b2b7aadc23d707c646d09f3b5e11421cebf22c2a..940dce2410ca3e950f365fa0150cb2aab23d3654 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -12,6 +12,9 @@
 
 ngIRCd CVSHEAD
 
+  - Results of the resolver (hostnames and IDENT names) are discarded after
+    the client is successfully registered with the server.
+  - Better logging while establishing and shutting down connections.
   - The type of service (TOS) of all sockets is set to "interactive" now.
   - Added short command line option "-t" as alternative to "--configtest".
   - Added optional support for "IDENT" lookups on incoming connections. You
@@ -501,4 +504,4 @@ ngIRCd 0.0.1, 31.12.2001
 
 
 -- 
-$Id: ChangeLog,v 1.226 2004/02/05 14:27:16 alex Exp $
+$Id: ChangeLog,v 1.227 2004/02/28 02:01:01 alex Exp $

diff --git a/src/ngircd/conn.c b/src/ngircd/conn.c
index 42ccd0022202ee8ea9f1858f710b8ffc4f853822..cea5c4bed55924888aa996f06a446ec1c7308e4d 100644 (file)
--- a/src/ngircd/conn.c
+++ b/src/ngircd/conn.c
@@ -1,6 +1,6 @@
 /*
  * ngIRCd -- The Next Generation IRC Daemon
- * Copyright (c)2001-2004 by Alexander Barton (alex@barton.de)
+ * Copyright (c)2001-2004 Alexander Barton <alex@barton.de>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -16,7 +16,7 @@
 
 #include "portab.h"
 
-static char UNUSED id[] = "$Id: conn.c,v 1.131 2004/02/03 20:28:30 alex Exp $";
+static char UNUSED id[] = "$Id: conn.c,v 1.132 2004/02/28 02:01:01 alex Exp $";
 
 #include "imp.h"
 #include <assert.h>
@@ -614,6 +614,7 @@ Conn_Close( CONN_ID Idx, CHAR *LogMsg, CHAR *FwdMsg, BOOLEAN InformClient )
         * sub-processes are closed down. */
 
        CLIENT *c;
+       CHAR *txt;
        DOUBLE in_k, out_k;
 #ifdef ZLIB
        DOUBLE in_z_k, out_z_k;
@@ -633,6 +634,12 @@ Conn_Close( CONN_ID Idx, CHAR *LogMsg, CHAR *FwdMsg, BOOLEAN InformClient )
 
        /* Mark link as "closing" */
        My_Connections[Idx].options |= CONN_ISCLOSING;
+               
+       if( LogMsg ) txt = LogMsg;
+       else txt = FwdMsg;
+       if( ! txt ) txt = "Reason unknown";
+
+       Log( LOG_INFO, "Shutting down connection %d (%s) with %s:%d ...", Idx, LogMsg ? LogMsg : FwdMsg, My_Connections[Idx].host, ntohs( My_Connections[Idx].addr.sin_port ));
 
        /* Search client, if any */
        c = Client_GetFromConn( Idx );
@@ -651,7 +658,6 @@ Conn_Close( CONN_ID Idx, CHAR *LogMsg, CHAR *FwdMsg, BOOLEAN InformClient )
                /* Send ERROR to client (see RFC!) */
                if( FwdMsg ) Conn_WriteStr( Idx, "ERROR :%s", FwdMsg );
                else Conn_WriteStr( Idx, "ERROR :Closing connection." );
-               if( My_Connections[Idx].sock == NONE ) return;
        }
 
        /* Try to write out the write buffer */
@@ -872,11 +878,9 @@ Handle_Write( CONN_ID Idx )
 
                        return FALSE;
                }
-#ifdef DEBUG
-               Log( LOG_DEBUG, "Connection %d with \"%s:%d\" established, now sendig PASS and SERVER ...", Idx, My_Connections[Idx].host, Conf_Server[Conf_GetServer( Idx )].port );
-#endif
+               Log( LOG_INFO, "Connection %d with \"%s:%d\" established. Now logging in ...", Idx, My_Connections[Idx].host, Conf_Server[Conf_GetServer( Idx )].port );
 
-               /* PASS und SERVER verschicken */
+               /* Send PASS and SERVER command to peer */
                Conn_WriteStr( Idx, "PASS %s %s", Conf_Server[Conf_GetServer( Idx )].pwd_out, NGIRCd_ProtoID );
                return Conn_WriteStr( Idx, "SERVER %s :%s", Conf_ServerName, Conf_ServerInfo );
        }
@@ -1643,12 +1647,25 @@ Read_Resolver_Result( INT r_fd )
 
        if( My_Connections[i].sock > NONE )
        {
+               /* Incoming connection */
 #ifdef IDENTAUTH
                CHAR *ident;
 #endif
-               /* Incoming connection: set hostname */
+
+               /* Search client ... */
                c = Client_GetFromConn( i );
                assert( c != NULL );
+
+               /* Only update client information of unregistered clients */
+               if( Client_Type( c ) != CLIENT_UNKNOWN )
+               {
+#ifdef DEBUG
+                       Log( LOG_DEBUG, "Resolver: discarding result for already registered connection %d.", i );
+#endif
+                       return;
+               }               
+
+               /* Set hostname */
                strlcpy( My_Connections[i].host, result, sizeof( My_Connections[i].host ));
                Client_SetHostname( c, result );
 
@@ -1656,21 +1673,21 @@ Read_Resolver_Result( INT r_fd )
                ident = strchr( result, 0 );
                ident++;
 
-               /* Do we have a result of the IDENT lookup? */
+               /* Do we have a result of the IDENT lookup? If so, set it as the user name */
                if( *ident )
                {
-                       Log( LOG_INFO, "IDENT lookup on connection %ld: \"%s\".", i, ident );
+                       Log( LOG_INFO, "IDENT lookup for connection %ld: \"%s\".", i, ident );
                        Client_SetUser( c, ident, TRUE );
                }
-               else Log( LOG_INFO, "IDENT lookup on connection %ld: no result.", i );
+               else Log( LOG_INFO, "IDENT lookup for connection %ld: no result.", i );
 #endif
        }
        else
        {
                /* Outgoing connection (server link!): set IP address */
                n = Conf_GetServer( i );
-               if( n > NONE ) strlcpy( Conf_Server[n].ip, result, sizeof( Conf_Server[n].ip ));
-               else Log( LOG_ERR, "Got resolver result for non-configured server!?" );
+               assert( n > NONE );
+               strlcpy( Conf_Server[n].ip, result, sizeof( Conf_Server[n].ip ));
        }
 
        /* Reset penalty time */