aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim/Framework/WebUtil.cs (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Fixed last-resort sending of error response (HTTP 500) when an error occurs ↵Oren Hurvitz2014-04-021-6/+6
| | | | | | while handling a request. The previous code didn't actually send the response, so the caller was stuck until the timeout (100 seconds).
* Fixed a case where logging an HTTP response failed because the stream was ↵Oren Hurvitz2014-03-271-23/+28
| | | | non-seekable
* Log errors in MakeRequest() as INFO, not ERROR. Some communications problems ↵Oren Hurvitz2014-03-271-2/+2
| | | | | | are benign (e.g., can't send Friend status update to a region that is down), so don't log them as ERROR so soon. We rethrow the exception, so the caller can still decide to log the error as an ERROR. Resolves http://opensimulator.org/mantis/view.php?id=7077
* Use the "X-Content-Encoding" header to indicate gzipped data, because old ↵Oren Hurvitz2014-03-251-1/+1
| | | | OpenSims fail if they get an unknown "Content-Encoding"
* Use the "Content-Encoding" header to indicate gzipped streamsOren Hurvitz2014-03-251-3/+5
|
* Always throw an exception if MakeRequest (used for HTTP POST) fails. ↵Oren Hurvitz2014-03-251-19/+10
| | | | | | (Previously many exceptions were ignored) Resolves http://opensimulator.org/mantis/view.php?id=6949
* Improved logging of HTTP requestsOren Hurvitz2014-03-251-35/+87
| | | | | | | | | | | - 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
* Set up a StreamReader and call ReadToEnd() instead of using the ↵Justin Clark-Casey (justincc)2014-02-271-41/+13
| | | | | | GetStreamString() extension method This eliminates some stream seeking that was never necessary and makes disposable of the StreamReader consistent with other code
* refactor: More consistently use using construct within WebUtil to match ↵Justin Clark-Casey (justincc)2014-02-271-225/+218
| | | | other existing code there.
* Add more details to InvalidOperationException logging in ↵Justin Clark-Casey (justincc)2014-02-271-3/+4
| | | | SychronousRestFormsRequest.MakeRequest().
* Make sure Web streams are disposed after useOren Hurvitz2014-01-311-3/+3
|
* refactor: replace verbose checks with String.IsNullOrEmpty where applicable.Justin Clark-Casey (justincc)2013-11-151-1/+1
| | | | Thanks to Kira for this patch from http://opensimulator.org/mantis/view.php?id=6845
* Make it possible to switch whether we serialize osd requests per endpoint or ↵Justin Clark-Casey (justincc)2013-08-051-4/+16
| | | | | | not, either via config (SerializeOSDRequests in [Network]) or via the "debug comms set" console command. For debug purposes to assess what impact this has on network response in a heavy test environment.
* Issue: 10 simultaneous TPs, many not making it. Now bypassing the per-url ↵Diva Canto2013-08-021-3/+3
| | | | lock -- we should be "ok" (or, more "ok") now that we have increased the connection limit on the http library. But this is a sensitive part of the code, so it may need reverting.
* Revert "WARNING: BRUTE FORCE DEBUG. AVOID USING THIS COMMIT."Diva Canto2013-07-041-4/+0
| | | | This reverts commit c95a23863ab51810ccc01afd3dd641c18a183305.
* WARNING: BRUTE FORCE DEBUG. AVOID USING THIS COMMIT.Diva Canto2013-07-041-0/+4
|
* Changed HG status notifications timeout down to 15secs from the default 100.Diva Canto2013-06-291-1/+9
|
* Implement logging of first 80 characters (debug level 5) or full body data ↵Justin Clark-Casey (justincc)2013-06-121-2/+52
| | | | | | | (debug level 6) on outgoing requests, depending on debug level This is set via "debug http out <level>" This matches the existing debug level behaviours for logging incoming http data
* Make sure we dispose of WebResponse, StreamReader and Stream in various ↵Justin Clark-Casey (justincc)2013-02-271-30/+31
| | | | places where we were not already.
* Allow setting connection limits, part 2Melanie2012-09-301-1/+26
|
* Revert "Added request.Proxy=null everywhere, as discussed in ↵Diva Canto2012-09-301-5/+0
| | | | | | | | http://stackoverflow.com/questions/2519655/httpwebrequest-is-extremely-slow." But the patch is here, in case anyone wants to try it. This reverts commit 531edd51d82ecd6a842a2611c99e9919634491ef.
* Added request.Proxy=null everywhere, as discussed in ↵Diva Canto2012-09-301-0/+5
| | | | | | http://stackoverflow.com/questions/2519655/httpwebrequest-is-extremely-slow. Thanks R.Gunther (rigun@rigutech.nl) https://lists.berlios.de/pipermail/opensim-users/2012-September/010986.html
* minor: Make slow outgoing request log messages consistent with other log ↵Justin Clark-Casey (justincc)2012-09-211-7/+7
| | | | messages
* Add request number counting to incoming HTTP requests in the same way that ↵Justin Clark-Casey (justincc)2012-09-211-1/+1
| | | | | | | | | 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.
* Insert a new log level 4 for HTTP IN and HTTP OUT that will log how long the ↵Justin Clark-Casey (justincc)2012-09-201-0/+24
| | | | | | | 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.
* Add ability to turn on/off logging of outgoing HTTP requests flowing through ↵Justin Clark-Casey (justincc)2012-09-201-8/+33
| | | | | | | | | 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.
* Change LongCallTime on WebUtil to 3000, to match the time where request ↵Justin Clark-Casey (justincc)2012-05-041-1/+1
| | | | | | | handling is considered "slow". This may be the wrong thing to do but stops lots of log spam in HG setups now that the monitoring is extended to other outgoing calls. LongCallTime may need to be made configurable.
* Extend 'slow' request logging to other server outbound requests (forms, ↵Justin Clark-Casey (justincc)2012-05-041-79/+194
| | | | | | rest, async rest) as well as the existing logging on outbound OSD requests. Also prints out the first 100 chars of any slow request data since this can contain useful info (such as agent ID).
* Add request verb and url to error messages in WebUtil that lack this.Justin Clark-Casey (justincc)2012-04-261-17/+38
| | | | Make exception printing consistent across windows and mono.
* Rename the stream extension method WebUtil.CopyTo() to WebUtil.CopyStream().Robert Adams2012-03-121-1/+6
| | | | | | | | | | .NET 4.0 added the method Stream.CopyTo(stream, bufferSize). For .NET 3.5 and before, WebUtil defined an extension method for Stream with the signature Stream.CopyTo(stream, maxBytesToCopy). The meaning of the second parameter is different in the two forms and depending on which compiler and/or runtime you use, you could get one form or the other. Crashes ensue. This change renames the WebUtil stream copy method to something that cannot be confused with the new CopyTo method defined in .NET 4.0.
* Serialize all web requests to a particular host:port pair; only applied to ↵Mic Bowman2012-03-091-0/+41
| | | | the PostToService variants.
* Some clean up in WebUtil, remove unused ServiceRequest function.Mic Bowman2012-02-271-71/+7
|
* remove some mono compiler warningsJustin Clark-Casey (justincc)2011-11-251-1/+1
|
* Get rid of the spurious [WEB UTIL] couldn't decode <OpenSim agent ↵Justin Clark-Casey (justincc)2011-11-191-1/+1
| | | | | | | 57956c4b-ff2e-4fc1-9995-613c6256cc98>: Invalid character 'O' in input string messages These are just the result of an attempt to canonicalize received messages - it's not important that we constantly log them. Also finally get the deregister grid service message working properly
* Make OpenSim.Framework.Servers.HttpServer rely on OpenSim.Framework instead ↵Justin Clark-Casey (justincc)2011-10-251-31/+29
| | | | | | | | | 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)
* minor: remove whitespaceJustin Clark-Casey (justincc)2011-07-131-1/+0
|
* Remove an unused 404 checker since on a 404, GetResponse() throws a ↵Justin Clark-Casey (justincc)2011-07-131-3/+0
| | | | WebException rather than proceeding.
* if a SynchronousRestObjectRequester request fails, then unless it's due to a ↵Justin Clark-Casey (justincc)2011-07-131-0/+11
| | | | | | | 404, log the exception for diagnostic purposes This is instead of logging "Invalid XML" for all failures, even if they weren't a result of invalid xml. A default TReponse is returned in the event of a 404, which is the same behaviour as previously.
* Remove the obsoleted SynchronousRestObjectPosterJustin Clark-Casey (justincc)2011-07-131-9/+0
|
* Changed yet more Warns to Debugs.Diva Canto2011-05-141-4/+4
|
* Also changed a couple of messages in WebUtil from Wanr/Info to Debug.Diva Canto2011-05-141-2/+2
|
* Stop spamming Nebadon's console when assets aren't found in the asset server.Diva Canto2011-05-141-1/+4
|
* Hopefully this fixes offline messages. The problem was: the server is not ↵Diva Canto2011-05-121-7/+2
| | | | setting the ContentLength of the response. That comes up to OpenSim as ContentLength=-1, which made the existing test fail.
* One more thing printed out. #OfflineIMDiva Canto2011-05-121-4/+1
|
* Instrument the heck out of offline messages. THIS IS VERY VERBOSE.Diva Canto2011-05-121-0/+16
|
* One more debug message for offline IMs.Diva Canto2011-05-121-1/+2
|
* Sequencing the using and try clauses as they were before, but this shows ↵Diva Canto2011-05-121-7/+7
| | | | that that obsolete function is not catching 404's as it should...
* Bummer, can't print the data I wanted to see. Printing just the context.Diva Canto2011-05-121-9/+1
|
* Tracking a problem with offline IMs coming in as null list.Diva Canto2011-05-121-6/+19
|
* This makes compression of fatpacks actually work. Previously they always ↵Diva Canto2011-05-111-7/+7
| | | | failed. See comment in WebUtil.