diff options
author | Justin Clark-Casey (justincc) | 2012-09-21 00:29:13 +0100 |
---|---|---|
committer | Justin Clark-Casey (justincc) | 2012-09-21 00:29:13 +0100 |
commit | 1b0abf8f0cd417e2a37cffc96379274ad98183f2 (patch) | |
tree | 9212648471fe15b0482e0a1fff835c43e7d81b2e /OpenSim/ApplicationPlugins | |
parent | Add request number counting to incoming HTTP requests in the same way that th... (diff) | |
download | opensim-SC_OLD-1b0abf8f0cd417e2a37cffc96379274ad98183f2.zip opensim-SC_OLD-1b0abf8f0cd417e2a37cffc96379274ad98183f2.tar.gz opensim-SC_OLD-1b0abf8f0cd417e2a37cffc96379274ad98183f2.tar.bz2 opensim-SC_OLD-1b0abf8f0cd417e2a37cffc96379274ad98183f2.tar.xz |
Comment out the long unused afaik HTTP agent handlers.
As far as I know, this was only used by the IBM Rest modules, much of which has been commented out for a very long time now. Other similar code uses HTTP or stream handlers instead.
So commenting this out to reduce code complexity and the need to make this facility consistent with the others where it may not be used anyway.
If this facility is actually being used then please notify me or uncomment it if you are core.
Diffstat (limited to '')
-rw-r--r-- | OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs | 30 | ||||
-rw-r--r-- | OpenSim/ApplicationPlugins/Rest/RestPlugin.cs | 14 |
2 files changed, 25 insertions, 19 deletions
diff --git a/OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs b/OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs index cb88695..072bd6f 100644 --- a/OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs +++ b/OpenSim/ApplicationPlugins/Rest/Inventory/RestHandler.cs | |||
@@ -312,14 +312,16 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory | |||
312 | // Now that everything is setup we can proceed to | 312 | // Now that everything is setup we can proceed to |
313 | // add THIS agent to the HTTP server's handler list | 313 | // add THIS agent to the HTTP server's handler list |
314 | 314 | ||
315 | if (!AddAgentHandler(Rest.Name,this)) | 315 | // FIXME: If this code is ever to be re-enabled (most of it is disabled already) then this will |
316 | { | 316 | // have to be handled through the AddHttpHandler interface. |
317 | Rest.Log.ErrorFormat("{0} Unable to activate handler interface", MsgId); | 317 | // if (!AddAgentHandler(Rest.Name,this)) |
318 | foreach (IRest handler in handlers) | 318 | // { |
319 | { | 319 | // Rest.Log.ErrorFormat("{0} Unable to activate handler interface", MsgId); |
320 | handler.Close(); | 320 | // foreach (IRest handler in handlers) |
321 | } | 321 | // { |
322 | } | 322 | // handler.Close(); |
323 | // } | ||
324 | // } | ||
323 | 325 | ||
324 | } | 326 | } |
325 | catch (Exception e) | 327 | catch (Exception e) |
@@ -342,11 +344,13 @@ namespace OpenSim.ApplicationPlugins.Rest.Inventory | |||
342 | { | 344 | { |
343 | Rest.Log.InfoFormat("{0} Plugin is terminating", MsgId); | 345 | Rest.Log.InfoFormat("{0} Plugin is terminating", MsgId); |
344 | 346 | ||
345 | try | 347 | // FIXME: If this code is ever to be re-enabled (most of it is disabled already) then this will |
346 | { | 348 | // have to be handled through the AddHttpHandler interface. |
347 | RemoveAgentHandler(Rest.Name, this); | 349 | // try |
348 | } | 350 | // { |
349 | catch (KeyNotFoundException){} | 351 | // RemoveAgentHandler(Rest.Name, this); |
352 | // } | ||
353 | // catch (KeyNotFoundException){} | ||
350 | 354 | ||
351 | foreach (IRest handler in handlers) | 355 | foreach (IRest handler in handlers) |
352 | { | 356 | { |
diff --git a/OpenSim/ApplicationPlugins/Rest/RestPlugin.cs b/OpenSim/ApplicationPlugins/Rest/RestPlugin.cs index eb16750..a2425b5 100644 --- a/OpenSim/ApplicationPlugins/Rest/RestPlugin.cs +++ b/OpenSim/ApplicationPlugins/Rest/RestPlugin.cs | |||
@@ -297,7 +297,9 @@ namespace OpenSim.ApplicationPlugins.Rest | |||
297 | { | 297 | { |
298 | if (!IsEnabled) return false; | 298 | if (!IsEnabled) return false; |
299 | _agents.Add(agentName, handler); | 299 | _agents.Add(agentName, handler); |
300 | return _httpd.AddAgentHandler(agentName, handler); | 300 | // return _httpd.AddAgentHandler(agentName, handler); |
301 | |||
302 | return false; | ||
301 | } | 303 | } |
302 | 304 | ||
303 | /// <summary> | 305 | /// <summary> |
@@ -316,7 +318,7 @@ namespace OpenSim.ApplicationPlugins.Rest | |||
316 | if (_agents[agentName] == handler) | 318 | if (_agents[agentName] == handler) |
317 | { | 319 | { |
318 | _agents.Remove(agentName); | 320 | _agents.Remove(agentName); |
319 | return _httpd.RemoveAgentHandler(agentName, handler); | 321 | // return _httpd.RemoveAgentHandler(agentName, handler); |
320 | } | 322 | } |
321 | return false; | 323 | return false; |
322 | } | 324 | } |
@@ -358,10 +360,10 @@ namespace OpenSim.ApplicationPlugins.Rest | |||
358 | _httpd.RemoveStreamHandler(h.HttpMethod, h.Path); | 360 | _httpd.RemoveStreamHandler(h.HttpMethod, h.Path); |
359 | } | 361 | } |
360 | _handlers = null; | 362 | _handlers = null; |
361 | foreach (KeyValuePair<string, IHttpAgentHandler> h in _agents) | 363 | // foreach (KeyValuePair<string, IHttpAgentHandler> h in _agents) |
362 | { | 364 | // { |
363 | _httpd.RemoveAgentHandler(h.Key, h.Value); | 365 | // _httpd.RemoveAgentHandler(h.Key, h.Value); |
364 | } | 366 | // } |
365 | _agents = null; | 367 | _agents = null; |
366 | } | 368 | } |
367 | 369 | ||