From bdc792d319601caa93790b21c33b3b623a4ac13c Mon Sep 17 00:00:00 2001 From: Dr Scofield Date: Thu, 22 May 2008 12:00:01 +0000 Subject: here are further enhancements to the IHttpAgentHandler and to BaseHttpServer (from awebb) i've added the OSHttpStatusCodes enumeration of HTTP status codes, have adapted BaseHttpServer to use those. then RestPlugin now has proper Failure handling returning proper HTTP status codes. Regions/POSTHandler is work-in-progress. --- .../ApplicationPlugins/Rest/Regions/GETHandler.cs | 34 ++++--- .../ApplicationPlugins/Rest/Regions/POSTHandler.cs | 101 +++++++++++++++++++++ .../Rest/Regions/RestRegionPlugin.cs | 1 + OpenSim/ApplicationPlugins/Rest/RestPlugin.cs | 60 ++++++++++-- 4 files changed, 172 insertions(+), 24 deletions(-) create mode 100644 OpenSim/ApplicationPlugins/Rest/Regions/POSTHandler.cs (limited to 'OpenSim/ApplicationPlugins/Rest') diff --git a/OpenSim/ApplicationPlugins/Rest/Regions/GETHandler.cs b/OpenSim/ApplicationPlugins/Rest/Regions/GETHandler.cs index a319a8b..b89976f 100644 --- a/OpenSim/ApplicationPlugins/Rest/Regions/GETHandler.cs +++ b/OpenSim/ApplicationPlugins/Rest/Regions/GETHandler.cs @@ -67,18 +67,18 @@ namespace OpenSim.ApplicationPlugins.Rest.Regions try { // param empty: regions list - if (String.IsNullOrEmpty(param)) return GetHandlerRegions(); + if (String.IsNullOrEmpty(param)) return GetHandlerRegions(httpResponse); // param not empty: specific region - return GetHandlerRegion(param); + return GetHandlerRegion(httpResponse, param); } catch (Exception e) { - return Failure("GET", e); + return Failure(httpResponse, OSHttpStatusCode.ServerErrorInternalError, "GET", e); } } - public string GetHandlerRegions() + public string GetHandlerRegions(OSHttpResponse httpResponse) { XmlWriter.WriteStartElement(String.Empty, "regions", String.Empty); foreach (Scene s in App.SceneManager.Scenes) @@ -105,7 +105,7 @@ namespace OpenSim.ApplicationPlugins.Rest.Regions return XmlWriterResult; } - public string GetHandlerRegion(string param) + public string GetHandlerRegion(OSHttpResponse httpResponse, string param) { // be resilient and don't get confused by a terminating '/' param = param.TrimEnd(new char[]{'/'}); @@ -118,7 +118,8 @@ namespace OpenSim.ApplicationPlugins.Rest.Regions Scene scene = null; App.SceneManager.TryGetScene(regionID, out scene); - if (null == scene) return Failure("GET", "cannot find region"); + if (null == scene) return Failure(httpResponse, OSHttpStatusCode.ClientErrorNotFound, + "GET", "cannot find region {0}", regionID.ToString()); RegionDetails details = new RegionDetails(scene.RegionInfo); @@ -143,25 +144,27 @@ namespace OpenSim.ApplicationPlugins.Rest.Regions switch (comps[1].ToLower()) { case "terrain": - return RegionTerrain(scene); + return RegionTerrain(httpResponse, scene); case "stats": - return RegionStats(scene); + return RegionStats(httpResponse, scene); case "prims": - return RegionPrims(scene); + return RegionPrims(httpResponse, scene); } } - return Failure("GET", "too many parameters"); + return Failure(httpResponse, OSHttpStatusCode.ClientErrorBadRequest, + "GET", "too many parameters {0}", param); } #endregion GET methods - protected string RegionTerrain(Scene scene) + protected string RegionTerrain(OSHttpResponse httpResponse, Scene scene) { - return Failure("GET", "terrain not implemented"); + return Failure(httpResponse, OSHttpStatusCode.ServerErrorNotImplemented, + "GET", "terrain not implemented"); } - protected string RegionStats(Scene scene) + protected string RegionStats(OSHttpResponse httpResponse, Scene scene) { int users = scene.GetAvatars().Count; int objects = scene.Entities.Count - users; @@ -182,9 +185,10 @@ namespace OpenSim.ApplicationPlugins.Rest.Regions return XmlWriterResult; } - protected string RegionPrims(Scene scene) + protected string RegionPrims(OSHttpResponse httpResponse, Scene scene) { - return Failure("GET", "prims not implemented"); + return Failure(httpResponse, OSHttpStatusCode.ServerErrorNotImplemented, + "GET", "prims not implemented"); } } } diff --git a/OpenSim/ApplicationPlugins/Rest/Regions/POSTHandler.cs b/OpenSim/ApplicationPlugins/Rest/Regions/POSTHandler.cs new file mode 100644 index 0000000..00fe0d2 --- /dev/null +++ b/OpenSim/ApplicationPlugins/Rest/Regions/POSTHandler.cs @@ -0,0 +1,101 @@ +/* +* Copyright (c) Contributors, http://opensimulator.org/ +* See CONTRIBUTORS.TXT for a full list of copyright holders. +* +* Redistribution and use in source and binary forms, with or without +* modification, are permitted provided that the following conditions are met: +* * Redistributions of source code must retain the above copyright +* notice, this list of conditions and the following disclaimer. +* * Redistributions in binary form must reproduce the above copyright +* notice, this list of conditions and the following disclaimer in the +* documentation and/or other materials provided with the distribution. +* * Neither the name of the OpenSim Project nor the +* names of its contributors may be used to endorse or promote products +* derived from this software without specific prior written permission. +* +* THIS SOFTWARE IS PROVIDED BY THE DEVELOPERS ``AS IS'' AND ANY +* EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED +* WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE +* DISCLAIMED. IN NO EVENT SHALL THE CONTRIBUTORS BE LIABLE FOR ANY +* DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES +* (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; +* LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND +* ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT +* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS +* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. +* +*/ + +using System; +using System.Threading; +using System.Collections; +using System.Collections.Generic; +using System.IO; +using System.Net; +using System.Reflection; +using System.Text.RegularExpressions; +using System.Timers; +using System.Xml; +using System.Xml.Serialization; +using libsecondlife; +using Mono.Addins; +using Nwc.XmlRpc; +using Nini.Config; +using OpenSim.Framework; +using OpenSim.Framework.Console; +using OpenSim.Framework.Servers; +using OpenSim.Framework.Communications; +using OpenSim.Region.Environment.Scenes; +using OpenSim.ApplicationPlugins.Rest; + +namespace OpenSim.ApplicationPlugins.Rest.Regions +{ + + public partial class RestRegionPlugin : RestPlugin + { + #region POST methods + public string PostHandler(string request, string path, string param, + OSHttpRequest httpRequest, OSHttpResponse httpResponse) + { + // foreach (string h in httpRequest.Headers.AllKeys) + // foreach (string v in httpRequest.Headers.GetValues(h)) + // m_log.DebugFormat("{0} IsGod: {1} -> {2}", MsgID, h, v); + + MsgID = RequestID; + m_log.DebugFormat("{0} POST path {1} param {2}", MsgID, path, param); + + try + { + // param empty: new region post + if (!IsGod(httpRequest)) + // XXX: this needs to be turned into a FailureUnauthorized(...) + return Failure(httpResponse, OSHttpStatusCode.ClientErrorUnauthorized, + "GET", "you are not god"); + + if (String.IsNullOrEmpty(param)) return CreateRegion(httpRequest, httpResponse); + + return Failure(httpResponse, OSHttpStatusCode.ClientErrorNotFound, + "POST", "url {0} not supported", param); + } + catch (Exception e) + { + return Failure(httpResponse, OSHttpStatusCode.ServerErrorInternalError, "POST", e); + } + } + + public string CreateRegion(OSHttpRequest request, OSHttpResponse response) + { + XmlWriter.WriteStartElement(String.Empty, "regions", String.Empty); + foreach (Scene s in App.SceneManager.Scenes) + { + XmlWriter.WriteStartElement(String.Empty, "uuid", String.Empty); + XmlWriter.WriteString(s.RegionInfo.RegionID.ToString()); + XmlWriter.WriteEndElement(); + } + XmlWriter.WriteEndElement(); + + return XmlWriterResult; + } + #endregion POST methods + } +} diff --git a/OpenSim/ApplicationPlugins/Rest/Regions/RestRegionPlugin.cs b/OpenSim/ApplicationPlugins/Rest/Regions/RestRegionPlugin.cs index 9b888fa..6d585a4 100644 --- a/OpenSim/ApplicationPlugins/Rest/Regions/RestRegionPlugin.cs +++ b/OpenSim/ApplicationPlugins/Rest/Regions/RestRegionPlugin.cs @@ -92,6 +92,7 @@ namespace OpenSim.ApplicationPlugins.Rest.Regions // add REST method handlers AddRestStreamHandler("GET", "/regions/", GetHandler); + AddRestStreamHandler("POST", "/regions/", PostHandler); } catch (Exception e) { diff --git a/OpenSim/ApplicationPlugins/Rest/RestPlugin.cs b/OpenSim/ApplicationPlugins/Rest/RestPlugin.cs index 4b8cdc1..f1ca83d 100644 --- a/OpenSim/ApplicationPlugins/Rest/RestPlugin.cs +++ b/OpenSim/ApplicationPlugins/Rest/RestPlugin.cs @@ -66,7 +66,7 @@ namespace OpenSim.ApplicationPlugins.Rest private string _prefix; // URL prefix below // which all REST URLs // are living - private StringWriter _sw = null; + private StringWriter _sw = null; private XmlTextWriter _xw = null; private string _godkey; @@ -240,7 +240,8 @@ namespace OpenSim.ApplicationPlugins.Rest } } - private List _handlers = new List(); + private List _handlers = new List(); + private Dictionary _agents = new Dictionary(); /// /// Add a REST stream handler to the underlying HTTP server. @@ -271,15 +272,39 @@ namespace OpenSim.ApplicationPlugins.Rest /// /// name of agent handler /// agent handler method - /// true when the plugin is disabled or the agent - /// handler could not be added.. + /// false when the plugin is disabled or the agent + /// handler could not be added. Any generated exceptions are + /// allowed to drop through to the caller, i.e. ArgumentException. + /// public bool AddAgentHandler(string agentName, IHttpAgentHandler handler) { if (!IsEnabled) return false; + _agents.Add(agentName, handler); return _httpd.AddAgentHandler(agentName, handler); } /// + /// Remove a powerful Agent handler from the underlying HTTP + /// server. + /// + /// name of agent handler + /// agent handler method + /// false when the plugin is disabled or the agent + /// handler could not be removed. Any generated exceptions are + /// allowed to drop through to the caller, i.e. KeyNotFound. + /// + public bool RemoveAgentHandler(string agentName, IHttpAgentHandler handler) + { + if (!IsEnabled) return false; + if(_agents[agentName] == handler) + { + _agents.Remove(agentName); + return _httpd.RemoveAgentHandler(agentName, handler); + } + return false; + } + + /// /// Check whether the HTTP request came from god; that is, is /// the god_key as configured in the config section supplied /// via X-OpenSim-Godkey? @@ -316,19 +341,30 @@ namespace OpenSim.ApplicationPlugins.Rest _httpd.RemoveStreamHandler(h.HttpMethod, h.Path); } _handlers = null; + foreach (KeyValuePair h in _agents) + { + _httpd.RemoveAgentHandler(h.Key,h.Value); + } + _agents = null; } /// /// Return a failure message. /// /// origin of the failure message - /// failure message /// This should probably set a return code as /// well. (?) - protected string Failure(string method, string message) + protected string Failure(OSHttpResponse response, OSHttpStatusCode status, + string method, string format, params string[] msg) { - m_log.ErrorFormat("{0} {1} failed: {2}", MsgID, method, message); - return String.Format("{0}", message); + string m = String.Format(format, msg); + + response.StatusCode = (int)status; + response.StatusDescription = m; + + m_log.ErrorFormat("{0} {1} failed: {2}", MsgID, method, m); + return String.Format("{0}", m); } /// @@ -338,8 +374,14 @@ namespace OpenSim.ApplicationPlugins.Rest /// exception causing the failure message /// This should probably set a return code as /// well. (?) - public string Failure(string method, Exception e) + public string Failure(OSHttpResponse response, OSHttpStatusCode status, + string method, Exception e) { + string m = String.Format("exception occurred: {0}", e.Message); + + response.StatusCode = (int)status; + response.StatusDescription = m; + m_log.DebugFormat("{0} {1} failed: {2}", MsgID, method, e.ToString()); m_log.ErrorFormat("{0} {1} failed: {2}", MsgID, method, e.Message); -- cgit v1.1