| Commit message (Collapse) | Author | Files | Lines |
|---|
|
can trigger an exception is the URL was being removed at the same time.
This involves three steps
1) Return gracefully in UrlModule.HttpRequestHandler() instead of throwing an exception when the url cannot be found in its index
2) Return true instead of false in HasEvents() if no matching request is found in the map. This call will only happen in the first place for raced requests.
3) Return a 404 in GetEvents() if the request is not in the index, rather than a blank 200 OK.
Many thanks to Tom Haines in http://opensimulator.org/mantis/view.php?id=6051 for doing some of the work on this.
|
|
would always return OK even if some other status code had been set
|
|
This allows one to see incoming requests as they happen.
This required making everything use the common MainServer class for registering and retrieving http servers, rather than duplicate structures.
|
|
logic.
This does require the server to be added before it is set as the main Instance
|
|
the main instance.
|
|
warn to be logged if we receive invalid xml for xmlrpc.
|
|
Accidentally make responseString null by default instead of String.Empty.
It needs to be something in case the XmlRpcRequest deserialize throws an exception due to bad xml (a failure which we silently swallow!)
|
|
slow handling of requests.
This is to avoid logging a 'slow' request when the source of delay is the viewer in processing a response.
This is not something we can do much about on the server end - it's server-side delay that we're interested in.
To ensure consistency, this commit also had to refactor and simplify inbound non-poll network request handling, though there should be no functional change.
IOSHttpResponse no longer exposes the Send() method, only classes in OpenSim.Framework.Servers.HttpServer should be doing this.
Only the GetTextureHandler was sending its own response. Now it leaves this to BaseHttpServer, like all other core handlers.
|
|
|
|
timeout is breached.
This alarm can then invoke this to log extra information.
This is used in LLUDPServer to show which client was being processed when incoming and outgoing udp watchdog alarms are triggered.
|
|
Signed-off-by: BlueWall <jamesh@bluewallgroup.com>
|
|
Signed-off-by: BlueWall <jamesh@bluewallgroup.com>
|
|
This adds name and description of the request handler to http request logging when DebugLevel >= 1
|
|
|
|
slow handling of inbound requests.
|
|
can relate a slow request to what the handler actually does and the agent it serves, if applicable.
This is most useful for capabilities where the url is not self-describing.
|
|
This reverts commit e31e7c68c8abfd61fed6dabac5403d8adf42ae87.
Applied for patch assessment and accidentally committed too early.
|
|
|
|
BaseHttpServer.HandleRequest()
|
|
|
|
On the first frame, all startup scene objects are added to the physics scene.
This can cause a considerable delay, so we don't start raising the alarm on scene loop timeouts until the second frame.
This commit also slightly changes the behaviour of timeout reporting.
Previously, a report was made for the very first timed out thread, ignoring all others until the next watchdog check.
Instead, we now report every timed out thread, though we still only do this once no matter how long the timeout.
|
|
It appears that this is entirely unnecessary since the poll http handlers are dealt with on a separate code path.
|
|
of OSHttpRequest/OSHttpResponse.
This is required for the substitution of different HTTP servers or the newer HttpServer.dll without having to commit to a particular implementation.
This is also required to write regression tests that involve the HTTP layer.
If you need to recompile, all you need to do is replace OSHttpRequest/OSHttpResponse references with IOSHttpRequest/IOSHttpResponse.
|
|
|
|
path.
Also simplifies debug levels to just 0 and 1
|
|
non-event queue inbound http requests to a simulator
|
|
|
|
|
|
today in 8a0a78c.
Also adds these to the watchdogs with very large timeouts (should really be infinite)
|
|
of the other way around.
This is necessary so that code in HttpServer can use framework facilities such as the thread watchdog for monitoring purposes.
Doing this shuffle meant that MainServer was moved into OpenSim/Framework/Servers
Also had to make OpenSim.Framework.Console rely on OpenSim.Framework rather than the other way around since it in turn relies on HttpServer
MainConsole and some new interfaces had to be moved into OpenSim/Framework to allow this. This can be reverted if parts of OpenSim.Framework stop relying on console presence (cheifly RegionInfo)
|
|
Remove() presumably does this check anyway since it just returns false if the key is not in the collection.
|
|
BaseHttpServer.RemovePollServerHTTPHandler() that stopped existing code in ClientClosed() from actually tearing down the poll handler
Actually doing the tear down appear to have no ill effects with region crossing and teleport.
|
|
allow attachments to be temporarily turned off.
This is for debugging purposes. Defaults to Attachments Enabled
|
|
|
|
|
|
|
|
|
|
|
|
BaseHttpServer.RemoveLLSDHandler()
|
|
|
|
|
|
|
|
some issues with the zoom level -- TBD.
|
|
Adding ssl support for "Out of Band" applications such as the remote
admin module or Robust services
|
|
See http://opensimulator.org/mantis/view.php?id=5336
It turns out that viewer 2 was upset by the lack of a response to viv_watcher.php. This would send it into a continuous login loop.
Viewer 1 was quite happy to ignore the lack of response.
This commit puts in the bare minimum 'OK' message in response to viv_watcher.php. This allows viewer 2 voice to connect and appears to work.
However, at some point we need to fill out the watcher response, whatever that is.
|
|
OpenSim.Framework -- just pasted them in WebUtil. This is so that code that uses the Service connectors don't need to include the HttpServer dll -- that was odd.
|
|
Fixes several spelling mistakes
|
|
|
|
request method and target are reported correctly and drop the txn id
as it's empty 99% of the time.
|
|
|
