4 files changed, 69 insertions, 24 deletions

diff --git a/OpenSim/Framework/Communications/LoginService.cs b/OpenSim/Framework/Communications/LoginService.cs
index be47258..3e3216f 100644
--- a/OpenSim/Framework/Communications/LoginService.cs
+++ b/OpenSim/Framework/Communications/LoginService.cs
@@ -73,7 +73,7 @@ namespace OpenSim.Framework.UserManagement
         }
 
         /// <summary>
-        /// Main user login function
+        /// Called when we receive the client's initial 
         /// </summary>
         /// <param name="request">The XMLRPC request</param>
         /// <returns>The response to send</returns>
@@ -94,25 +94,32 @@ namespace OpenSim.Framework.UserManagement
 
                 string startLocationRequest = "last";
 
-                if (requestData.Contains("start"))
-                {
-                    startLocationRequest = (string)requestData["start"];
-                    m_log.Info("[LOGIN]: Client Requested Start: " + (string)requestData["start"]);
-                }
-
                 UserProfileData userProfile;
                 LoginResponse logResponse = new LoginResponse();
+                
+                string firstname = String.Empty;
+                string lastname = String.Empty;
 
                 if (GoodXML)
                 {
-                    string firstname = (string) requestData["first"];
-                    string lastname = (string) requestData["last"];
+                    firstname = (string) requestData["first"];
+                    lastname = (string) requestData["last"];
+                    
+                    m_log.InfoFormat(
+                         "[LOGIN]: Received login request message from user {0} {1}", 
+                         firstname, lastname);
 
                     if( requestData.Contains("version"))
                     {
                         string clientversion = (string)requestData["version"];
-                        m_log.Info("[LOGIN]: Client Version " + clientversion + " for " + firstname + " " + lastname);
+                        m_log.Info("[LOGIN]: Client version: " + clientversion);
                     }
+                    
+                    if (requestData.Contains("start"))
+                    {
+                        startLocationRequest = (string)requestData["start"];
+                        m_log.Info("[LOGIN]: Client requested start location: " + (string)requestData["start"]);
+                    }                    
 
                     userProfile = GetTheUser(firstname, lastname);
                     if (userProfile == null)
@@ -134,31 +141,35 @@ namespace OpenSim.Framework.UserManagement
                         {
                             webloginkey = new LLUUID((string)requestData["web_login_key"]);
                         }
-                        catch (System.Exception)
+                        catch (System.Exception e)
                         {
+                            m_log.InfoFormat(
+                                 "[LOGIN]: Bad web_login_key: {0} for user {1} {2}, exception {3}", 
+                                 requestData["web_login_key"], firstname, lastname, e);
+                            
                             return logResponse.CreateFailedResponse();
                         }
                         GoodLogin = AuthenticateUser(userProfile, webloginkey);
 
                     }
-                    else
-                    {
-                        return logResponse.CreateFailedResponse();
-                    }
                 }
                 else
                 {
+                    m_log.Info(
+                        "[LOGIN]: login_to_simulator login message did not contain all the required data");
+                    
                     return logResponse.CreateGridErrorResponse();
                 }
 
                 if (!GoodLogin)
                 {
+                    m_log.InfoFormat("[LOGIN]: User {0} {1} failed authentication", firstname, lastname);
+                    
                     return logResponse.CreateLoginFailedResponse();
                 }
                 else
                 {
                     // If we already have a session...
-
                     if (userProfile.currentAgent != null && userProfile.currentAgent.agentOnline)
                     {
                         //TODO: The following statements can cause trouble:
@@ -169,6 +180,11 @@ namespace OpenSim.Framework.UserManagement
                         m_userManager.CommitAgent(ref userProfile);
 
                         // Reject the login
+                        
+                        m_log.InfoFormat(
+                             "[LOGIN]: Notifying user {0} {1} that they are already logged in", 
+                             firstname, lastname);
+                        
                         return logResponse.CreateAlreadyLoggedInResponse();
                     }
                     // Otherwise...
@@ -234,14 +250,19 @@ namespace OpenSim.Framework.UserManagement
                         if (StatsManager.UserStats != null)
                             StatsManager.UserStats.AddSuccessfulLogin();
                         
+                        m_log.InfoFormat(
+                             "[LOGIN]: Authentication of user {0} {1} successful.  Sending response to client.",
+                             firstname, lastname);
+                        
                         return logResponse.ToXmlRpcResponse();
                     }
                     catch (Exception e)
                     {
-                        m_log.Info("[LOGIN]: " + e.ToString());
+                        m_log.Info("[LOGIN]: Login failed, exception" + e.ToString());
                     }
-                    //}
                 }
+
+                m_log.Info("[LOGIN]: Login failed.  Sending back blank XMLRPC response");
                 return response;
             }
             finally
diff --git a/OpenSim/Framework/NetworkServersInfo.cs b/OpenSim/Framework/NetworkServersInfo.cs
index 20cd768..df559fc 100644
--- a/OpenSim/Framework/NetworkServersInfo.cs
+++ b/OpenSim/Framework/NetworkServersInfo.cs
@@ -45,7 +45,7 @@ namespace OpenSim.Framework
 
         public string InventoryURL = String.Empty;
 
-        public static uint DefaultHttpListenerPort = 9000;
+        public static readonly uint DefaultHttpListenerPort = 9000;
         public uint HttpListenerPort = DefaultHttpListenerPort;
 
         public static uint RemotingListenerPort = 8895;
diff --git a/OpenSim/Framework/RegionInfo.cs b/OpenSim/Framework/RegionInfo.cs
index b5d6869..fc6da57 100644
--- a/OpenSim/Framework/RegionInfo.cs
+++ b/OpenSim/Framework/RegionInfo.cs
@@ -71,6 +71,7 @@ namespace OpenSim.Framework
             m_internalEndPoint = ConvertFrom.InternalEndPoint;
             m_externalHostName = ConvertFrom.ExternalHostName;
             m_remotingPort = ConvertFrom.RemotingPort;
+            m_httpPort = ConvertFrom.HttpPort;
             m_allow_alternate_ports = ConvertFrom.m_allow_alternate_ports;
             RemotingAddress = ConvertFrom.RemotingAddress;
             RegionID = LLUUID.Zero;
@@ -79,16 +80,27 @@ namespace OpenSim.Framework
 
         public LLUUID RegionID = LLUUID.Zero;
 
-        public uint m_remotingPort;
+        protected uint m_remotingPort;
 
         public uint RemotingPort
         {
             get { return m_remotingPort; }
             set { m_remotingPort = value; }
         }
+        
+        /// <value>
+        /// The port by which http communication occurs with the region (most noticeably, CAPS communication)
+        /// </value>
+        protected uint m_httpPort;
+        public uint HttpPort
+        {
+            get { return m_httpPort; }
+            set { m_httpPort = value; }
+        }
+        
         public bool m_allow_alternate_ports;
 
-        public string m_serverURI;
+        protected string m_serverURI;
         public string ServerURI
         {
             get
@@ -142,7 +154,6 @@ namespace OpenSim.Framework
         }
 
         protected string m_externalHostName;
-
         public string ExternalHostName
         {
             get { return m_externalHostName; }
diff --git a/OpenSim/Framework/SerializableRegionInfo.cs b/OpenSim/Framework/SerializableRegionInfo.cs
index 39a5993..2c73da8 100644
--- a/OpenSim/Framework/SerializableRegionInfo.cs
+++ b/OpenSim/Framework/SerializableRegionInfo.cs
@@ -49,6 +49,7 @@ namespace OpenSim.Framework
             m_internalEndPoint = ConvertFrom.InternalEndPoint;
             m_externalHostName = ConvertFrom.ExternalHostName;
             m_remotingPort = ConvertFrom.RemotingPort;
+            m_httpPort = ConvertFrom.HttpPort;
             m_allow_alternate_ports = ConvertFrom.m_allow_alternate_ports;
             RemotingAddress = ConvertFrom.RemotingAddress;
             m_proxyUrl = ConvertFrom.proxyUrl; 
@@ -79,17 +80,29 @@ namespace OpenSim.Framework
         public Guid RegionID = LLUUID.Zero.UUID;
 
         public uint m_remotingPort;
-
         public uint RemotingPort
         {
             get { return m_remotingPort; }
             set { m_remotingPort = value; }
         }
+        
+        /// <value>
+        /// The port by which http communication occurs with the region (most noticeably, CAPS communication)
+        /// 
+        /// FIXME: Defaulting to 9000 temporarily (on the basis that this is the http port most region 
+        /// servers are running) until the revision in which this change is made propogates around grids.
+        /// </value>
+        protected uint m_httpPort = 9000;
+        public uint HttpPort
+        {
+            get { return m_httpPort; }
+            set { m_httpPort = value; }
+        }        
+        
         public bool m_allow_alternate_ports;
 
         public string RemotingAddress;
 
-
         public IPEndPoint ExternalEndPoint
         {
             get