| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
|
|
|
|
|
| |
- MemoryBuffer isn't seekable, so we can't log it. Log the string instead.
- Handle compressed streams
- Don't attempt to dump binary data. Either don't log it at all (if we know it's binary), or at least convert non-ASCII characters to ASCII.
- Log responses to HTTP requests
- Use the same log prefix for all of these log messages ("[LOGHTTP]"), to make them easy to see at a glance
- Increased the snippet length to 200 (80 doesn't show enough), and add "..." only if the message was actually truncated
Resolves http://opensimulator.org/mantis/view.php?id=6949
|
| |
|
|
| |
Involved some restructuring to allow regression tests to dequeue inventory requests and perform poll responses synchronously rather than async
|
| |
|
|
| |
Thanks to Kira for this patch from http://opensimulator.org/mantis/view.php?id=6845
|
| |
|
|
| |
(cherry picked from commit 93abcde69043b175071e0bb752538d9730433f1d)
|
| |
|
|
|
|
|
| |
handlers.
This adds explicit cap poll handler supporting to the Caps classes rather than relying on callers to do the complicated coding.
Other refactoring was required to get logic into the right places to support this.
|
| |
|
|
| |
This happens in HEAD handlers.
|
| |
|
|
| |
server.network.HTTPRequestsMade in "show stats all"
|
| |
|
|
| |
httpserver.<port>.IncomingHTTPRequestsProcessed stat
|
| |
|
|
|
|
|
|
| |
in 3eee991 but removed in 7c0bfca
Do not rely on destructors to stop things.
These fire at unpredictable times and cause problems such as http://opensimulator.org/mantis/view.php?id=6503
and most probably http://opensimulator.org/mantis/view.php?id=6668
|
| | |
|
| | |
|
| |
|
|
| |
connection close issue by getting rid of the socket references * This adds a connection timeout checker to shutdown poor or evil connections and combats DOS attempts that just connect and make no complete requests and just wait. It also actually implements KeepAlive... instead of just understanding the connection header in the request... you can test by connecting and requesting a keepalive header and sending another request on the same connection. The new timeout checker closes expired keepalive sessions, just make sure you send the request within 70 seconds of connecting or the timeout checker will timeout the connection.
|
| | |
|
| | |
|
| |
|
|
| |
working.. Still have to do Header.ToBytes and compose a websocket frame with a payload.
|
| |\ |
|
| | |
| |
| |
| |
| |
| | |
destructor.
Hopes to address occasional shutdown failures from http://opensimulator.org/mantis/view.php?id=6503
|
| |/
|
|
| |
Adding additional return status for JsonRpcMethod. Now returns true/false
|
| |
|
|
|
| |
Added registration json-rpc handlers in the http server. Covers version 2.0
See: http://www.jsonrpc.org/specification
|
| |
|
|
| |
than creating a new CultureInfo separately
|
| | |
|
| |
|
|
| |
the end of the body.
|
| |
|
|
| |
instead of blank if no content type was set.
|
| |
|
|
|
|
| |
where this wasn't already used.
This allows log level 5 (log sample or large part of incoming post data) to operate and removes copy/paste.
|
| |
|
|
|
|
| |
check we received a request handler first since this is not guaranteed.
Resolves harmless logged exception when content type and generic xml rpc requests take more than 3 seconds.
|
| | |
|
| |
|
|
| |
newlines at this level to '\n' to enable them to be logged.
|
| | |
|
| |
|
|
|
|
| |
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.
|
| |
|
|
|
|
|
|
|
| |
this was already being done for outgoing HTTP requests.
This allows us to associate debug logging messages with the right request.
It also allows us to put a request number on 'long request' logging even if other debug logging is not enabled, which gives us some idea of whether every request is suffering this problem or only some.
This is a separate internal number not associated with any incoming number in the opensim-request-id header, this will be clarified when logging of this incoming request number is re-enabled.
This commit also adds port number to HTTP IN logging to allow us to distinguish between different request numbers on different ports.
|
| |
|
|
|
|
|
| |
request took.
This is only printed if debug http level >= 4 and the request didn't take more than the time considered 'long', in which case the existing log message is printed.
This displaces the previous log levels 4 and 5 which are now 5 and 6 respectively.
|
| |
|
|
|
|
|
|
|
| |
WebUtil.
This is for debugging purposes.
This is controlled via the "debug http" command which can already log incoming requests.
This now gains a mandatory parameter of in, out or all to control what is logged.
Log messages are also shortened and labelled and HTTP IN or HTTP OUT to be consistent with existing UDP PACKET IN and PACKET OUT messages.
|
| |
|
|
| |
Also converts newlines to "\n" text.
|
| |
|
|
|
|
| |
incoming request data and the entire incoming data respectively.
See "help debug http" for more details.
|
| |
|
|
| |
with other monitoring code from OpenSim.Framework
|
| |
|
|
|
|
| |
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.
|
| |
|
|
| |
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()
|
| | |
|
| |
|
|
| |
It appears that this is entirely unnecessary since the poll http handlers are dealt with on a separate code path.
|
