From: Alan Webb <alan_webb@us.ibm.com>

  Fixed problem with REST services caused by changes to the OpenSim
  core code base - the comms manager had been 'modularized'.

  Also added additional debugging to RemoteAdmin interface.

5 files changed, 37 insertions, 42 deletions

diff --git a/OpenSim/ApplicationPlugins/Rest/Inventory/Rest.cs b/OpenSim/ApplicationPlugins/Rest/Inventory/Rest.cs
index 8ab9fd2..62e18c4 100644
--- a/OpenSim/ApplicationPlugins/Rest/Inventory/Rest.cs
+++ b/OpenSim/ApplicationPlugins/Rest/Inventory/Rest.cs
@@ -69,11 +69,6 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory
 
         internal static IRestHandler              Plugin            = null;
         internal static OpenSimBase               main              = null;
-        internal static CommunicationsManager     Comms             = null;
-        internal static IInventoryServices        InventoryServices = null;
-        internal static IUserService              UserServices      = null;
-        internal static IAvatarService            AvatarServices    = null;
-        internal static IAssetCache               AssetServices     = null;
         internal static string                    Prefix            = null;
         internal static IConfig                   Config            = null;
         internal static string                    GodKey            = null;
@@ -88,6 +83,39 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory
         internal static int                       DumpLineSize      = 32; // Should be a multiple of 16 or (possibly) 4
 
         /// <summary>
+        /// These are all dependent upon the Comms manager
+        /// being initialized. So they have to be properties
+        /// because the comms manager is now a module and is
+        /// not guaranteed to be there when the rest handler
+        /// initializes.
+        /// </summary>
+
+        internal static CommunicationsManager Comms
+        {
+            get { return main.CommunicationsManager; }
+        }
+
+        internal static IInventoryServices InventoryServices
+        {
+            get { return Comms.InventoryService; }
+        }
+
+        internal static IUserService UserServices
+        {
+            get { return Comms.UserService; }
+        }
+ 
+        internal static IAvatarService AvatarServices
+        {
+            get { return Comms.AvatarService; }
+        }
+
+        internal static IAssetCache AssetServices
+        {
+            get { return Comms.AssetCache; }
+        }
+
+        /// <summary>
         /// HTTP requires that status information be generated for PUT
         /// and POST opertaions. This is in support of that. The
         /// operation verb gets substituted into the first string,
diff --git a/OpenSim/ApplicationPlugins/Rest/Inventory/RestAppearanceServices.cs b/OpenSim/ApplicationPlugins/Rest/Inventory/RestAppearanceServices.cs
index 667cf57..fad0438 100644
--- a/OpenSim/ApplicationPlugins/Rest/Inventory/RestAppearanceServices.cs
+++ b/OpenSim/ApplicationPlugins/Rest/Inventory/RestAppearanceServices.cs
@@ -55,16 +55,6 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory
             Rest.Log.InfoFormat("{0} User appearance services initializing", MsgId);
             Rest.Log.InfoFormat("{0} Using REST Implementation Version {1}", MsgId, Rest.Version);
 
-            // This is better than a null reference.
-
-            if (Rest.AvatarServices == null)
-                throw new Exception(String.Format("{0} OpenSim inventory services are not available",
-                                                  MsgId));
-
-            if (Rest.UserServices == null)
-                throw new Exception(String.Format("{0} OpenSim user profile services are not available",
-                                                  MsgId));
-
             // If a relative path was specified for the handler's domain,
             // add the standard prefix to make it absolute, e.g. /admin
 
diff --git a/OpenSim/ApplicationPlugins/Rest/Inventory/RestAssetServices.cs b/OpenSim/ApplicationPlugins/Rest/Inventory/RestAssetServices.cs
index 3475d32..043fca8 100644
--- a/OpenSim/ApplicationPlugins/Rest/Inventory/RestAssetServices.cs
+++ b/OpenSim/ApplicationPlugins/Rest/Inventory/RestAssetServices.cs
@@ -46,12 +46,6 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory
             Rest.Log.InfoFormat("{0} Asset services initializing", MsgId);
             Rest.Log.InfoFormat("{0} Using REST Implementation Version {1}", MsgId, Rest.Version);
 
-            // This is better than a null reference.
-
-            if (Rest.AssetServices == null)
-                throw new Exception(String.Format("{0} OpenSim asset services are not available",
-                                                  MsgId));
-
             // If the handler specifies a relative path for its domain
             // then we must add the standard absolute prefix, e.g. /admin
 
diff --git a/OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs b/OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs
index f0a5308..5ff9da3 100644
--- a/OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs
+++ b/OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs
@@ -227,16 +227,13 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory
                 // them easy to reach from anywhere in the assembly.
 
                 Rest.main              = openSim;
+                if(Rest.main == null)
+                    throw new Exception("OpenSim base pointer is null");
+
                 Rest.Plugin            = this;
-                Rest.Comms             = Rest.main.CommunicationsManager;
-                Rest.UserServices      = Rest.Comms.UserService;
-                Rest.InventoryServices = Rest.Comms.InventoryService;
-                Rest.AssetServices     = Rest.Comms.AssetCache;
-                Rest.AvatarServices    = Rest.Comms.AvatarService;
                 Rest.Config            = Config;
                 Rest.Prefix            = Prefix;
                 Rest.GodKey            = GodKey;
-
                 Rest.Authenticate      = Rest.Config.GetBoolean("authenticate", Rest.Authenticate);
                 Rest.Scheme            = Rest.Config.GetString("auth-scheme", Rest.Scheme);
                 Rest.Secure            = Rest.Config.GetBoolean("secured", Rest.Secure);
@@ -253,7 +250,7 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory
                                     (Rest.Authenticate ? "" : "not "));
 
                 Rest.Log.InfoFormat("{0} Security is {1}enabled", MsgId,
-                                    (Rest.Authenticate ? "" : "not "));
+                                    (Rest.Secure ? "" : "not "));
 
                 Rest.Log.InfoFormat("{0} Extended URI escape processing is {1}enabled", MsgId,
                                     (Rest.ExtendedEscape ? "" : "not "));
diff --git a/OpenSim/ApplicationPlugins/Rest/Inventory/RestInventoryServices.cs b/OpenSim/ApplicationPlugins/Rest/Inventory/RestInventoryServices.cs
index 9935046..95e3d71 100644
--- a/OpenSim/ApplicationPlugins/Rest/Inventory/RestInventoryServices.cs
+++ b/OpenSim/ApplicationPlugins/Rest/Inventory/RestInventoryServices.cs
@@ -62,20 +62,6 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory
             Rest.Log.InfoFormat("{0} Inventory services initializing", MsgId);
             Rest.Log.InfoFormat("{0} Using REST Implementation Version {1}", MsgId, Rest.Version);
 
-            // This is better than a null reference.
-
-            if (Rest.InventoryServices == null)
-                throw new Exception(String.Format("{0} OpenSim inventory services are not available",
-                        MsgId));
-
-            if (Rest.UserServices == null)
-                throw new Exception(String.Format("{0} OpenSim user services are not available",
-                        MsgId));
-
-            if (Rest.AssetServices == null)
-                throw new Exception(String.Format("{0} OpenSim asset services are not available",
-                        MsgId));
-
             // If a relative path was specified for the handler's domain,
             // add the standard prefix to make it absolute, e.g. /admin