diff options
author | BlueWall | 2012-04-26 16:40:36 -0400 |
---|---|---|
committer | BlueWall | 2012-04-26 16:40:36 -0400 |
commit | 2542ca2a49d70134394ce23c2594b26668d3a8b5 (patch) | |
tree | 44c6eee8a3b64571ed64494c238b7308b1509990 | |
parent | Merge branch 'master' of /home/opensim/var/repo/opensim (diff) | |
parent | Add request verb and url to error messages in WebUtil that lack this. (diff) | |
download | opensim-SC-2542ca2a49d70134394ce23c2594b26668d3a8b5.zip opensim-SC-2542ca2a49d70134394ce23c2594b26668d3a8b5.tar.gz opensim-SC-2542ca2a49d70134394ce23c2594b26668d3a8b5.tar.bz2 opensim-SC-2542ca2a49d70134394ce23c2594b26668d3a8b5.tar.xz |
Merge branch 'master' of ssh://opensimulator.org/var/git/opensim
-rw-r--r-- | OpenSim/Framework/WebUtil.cs | 55 | ||||
-rw-r--r-- | OpenSim/Region/Framework/Scenes/Scene.cs | 4 | ||||
-rw-r--r-- | OpenSim/Region/Framework/Scenes/ScenePresence.cs | 6 |
3 files changed, 43 insertions, 22 deletions
diff --git a/OpenSim/Framework/WebUtil.cs b/OpenSim/Framework/WebUtil.cs index ead8f46..d2aa538 100644 --- a/OpenSim/Framework/WebUtil.cs +++ b/OpenSim/Framework/WebUtil.cs | |||
@@ -86,8 +86,7 @@ namespace OpenSim.Framework | |||
86 | return eplock; | 86 | return eplock; |
87 | } | 87 | } |
88 | } | 88 | } |
89 | 89 | ||
90 | |||
91 | #region JSONRequest | 90 | #region JSONRequest |
92 | 91 | ||
93 | /// <summary> | 92 | /// <summary> |
@@ -216,7 +215,9 @@ namespace OpenSim.Framework | |||
216 | reqnum,url,method,tickdiff,tickdata); | 215 | reqnum,url,method,tickdiff,tickdata); |
217 | } | 216 | } |
218 | 217 | ||
219 | m_log.DebugFormat("[WEB UTIL]: <{0}> osd request for {1}, method {2} FAILED: {3}", reqnum, url, method, errorMessage); | 218 | m_log.DebugFormat( |
219 | "[WEB UTIL]: <{0}> osd request for {1}, method {2} FAILED: {3}", reqnum, url, method, errorMessage); | ||
220 | |||
220 | return ErrorResponseMap(errorMessage); | 221 | return ErrorResponseMap(errorMessage); |
221 | } | 222 | } |
222 | 223 | ||
@@ -357,7 +358,8 @@ namespace OpenSim.Framework | |||
357 | reqnum,url,method,tickdiff,tickdata); | 358 | reqnum,url,method,tickdiff,tickdata); |
358 | } | 359 | } |
359 | 360 | ||
360 | m_log.WarnFormat("[WEB UTIL]: <{0}> form request failed: {1}",reqnum,errorMessage); | 361 | m_log.WarnFormat("[WEB UTIL]: <{0}> form request to {1} failed: {2}", reqnum, url, errorMessage); |
362 | |||
361 | return ErrorResponseMap(errorMessage); | 363 | return ErrorResponseMap(errorMessage); |
362 | } | 364 | } |
363 | 365 | ||
@@ -771,12 +773,16 @@ namespace OpenSim.Framework | |||
771 | } | 773 | } |
772 | else | 774 | else |
773 | { | 775 | { |
774 | m_log.ErrorFormat("[ASYNC REQUEST]: Request {0} {1} failed with status {2} and message {3}", verb, requestUrl, e.Status, e.Message); | 776 | m_log.ErrorFormat( |
777 | "[ASYNC REQUEST]: Request {0} {1} failed with status {2} and message {3}", | ||
778 | verb, requestUrl, e.Status, e.Message); | ||
775 | } | 779 | } |
776 | } | 780 | } |
777 | catch (Exception e) | 781 | catch (Exception e) |
778 | { | 782 | { |
779 | m_log.ErrorFormat("[ASYNC REQUEST]: Request {0} {1} failed with exception {2}", verb, requestUrl, e); | 783 | m_log.ErrorFormat( |
784 | "[ASYNC REQUEST]: Request {0} {1} failed with exception {2}{3}", | ||
785 | verb, requestUrl, e.Message, e.StackTrace); | ||
780 | } | 786 | } |
781 | 787 | ||
782 | // m_log.DebugFormat("[ASYNC REQUEST]: Received {0}", deserial.ToString()); | 788 | // m_log.DebugFormat("[ASYNC REQUEST]: Received {0}", deserial.ToString()); |
@@ -788,7 +794,8 @@ namespace OpenSim.Framework | |||
788 | catch (Exception e) | 794 | catch (Exception e) |
789 | { | 795 | { |
790 | m_log.ErrorFormat( | 796 | m_log.ErrorFormat( |
791 | "[ASYNC REQUEST]: Request {0} {1} callback failed with exception {2}", verb, requestUrl, e); | 797 | "[ASYNC REQUEST]: Request {0} {1} callback failed with exception {2}{3}", |
798 | verb, requestUrl, e.Message, e.StackTrace); | ||
792 | } | 799 | } |
793 | 800 | ||
794 | }, null); | 801 | }, null); |
@@ -841,7 +848,8 @@ namespace OpenSim.Framework | |||
841 | } | 848 | } |
842 | catch (Exception e) | 849 | catch (Exception e) |
843 | { | 850 | { |
844 | m_log.DebugFormat("[FORMS]: exception occured on sending request to {0}: " + e.ToString(), requestUrl); | 851 | m_log.DebugFormat( |
852 | "[FORMS]: exception occured {0} {1}: {2}{3}", verb, requestUrl, e.Message, e.StackTrace); | ||
845 | } | 853 | } |
846 | finally | 854 | finally |
847 | { | 855 | { |
@@ -867,7 +875,9 @@ namespace OpenSim.Framework | |||
867 | } | 875 | } |
868 | catch (Exception e) | 876 | catch (Exception e) |
869 | { | 877 | { |
870 | m_log.DebugFormat("[FORMS]: exception occured on receiving reply " + e.ToString()); | 878 | m_log.DebugFormat( |
879 | "[FORMS]: Exception occured on receiving {0} {1}: {2}{3}", | ||
880 | verb, requestUrl, e.Message, e.StackTrace); | ||
871 | } | 881 | } |
872 | finally | 882 | finally |
873 | { | 883 | { |
@@ -880,7 +890,7 @@ namespace OpenSim.Framework | |||
880 | catch (System.InvalidOperationException) | 890 | catch (System.InvalidOperationException) |
881 | { | 891 | { |
882 | // This is what happens when there is invalid XML | 892 | // This is what happens when there is invalid XML |
883 | m_log.DebugFormat("[FORMS]: InvalidOperationException on receiving request"); | 893 | m_log.DebugFormat("[FORMS]: InvalidOperationException on receiving {0} {1}", verb, requestUrl); |
884 | } | 894 | } |
885 | } | 895 | } |
886 | return respstring; | 896 | return respstring; |
@@ -938,7 +948,10 @@ namespace OpenSim.Framework | |||
938 | } | 948 | } |
939 | catch (Exception e) | 949 | catch (Exception e) |
940 | { | 950 | { |
941 | m_log.DebugFormat("[SynchronousRestObjectRequester]: exception in sending data to {0}: {1}", requestUrl, e); | 951 | m_log.DebugFormat( |
952 | "[SynchronousRestObjectRequester]: Exception in making request {0} {1}: {2}{3}", | ||
953 | verb, requestUrl, e.Message, e.StackTrace); | ||
954 | |||
942 | return deserial; | 955 | return deserial; |
943 | } | 956 | } |
944 | finally | 957 | finally |
@@ -960,7 +973,11 @@ namespace OpenSim.Framework | |||
960 | respStream.Close(); | 973 | respStream.Close(); |
961 | } | 974 | } |
962 | else | 975 | else |
963 | m_log.DebugFormat("[SynchronousRestObjectRequester]: Oops! no content found in response stream from {0} {1}", requestUrl, verb); | 976 | { |
977 | m_log.DebugFormat( | ||
978 | "[SynchronousRestObjectRequester]: Oops! no content found in response stream from {0} {1}", | ||
979 | verb, requestUrl); | ||
980 | } | ||
964 | } | 981 | } |
965 | } | 982 | } |
966 | catch (WebException e) | 983 | catch (WebException e) |
@@ -971,20 +988,24 @@ namespace OpenSim.Framework | |||
971 | return deserial; | 988 | return deserial; |
972 | else | 989 | else |
973 | m_log.ErrorFormat( | 990 | m_log.ErrorFormat( |
974 | "[SynchronousRestObjectRequester]: WebException {0} {1} {2} {3}", | 991 | "[SynchronousRestObjectRequester]: WebException for {0} {1} {2}: {3} {4}", |
975 | requestUrl, typeof(TResponse).ToString(), e.Message, e.StackTrace); | 992 | verb, requestUrl, typeof(TResponse).ToString(), e.Message, e.StackTrace); |
976 | } | 993 | } |
977 | catch (System.InvalidOperationException) | 994 | catch (System.InvalidOperationException) |
978 | { | 995 | { |
979 | // This is what happens when there is invalid XML | 996 | // This is what happens when there is invalid XML |
980 | m_log.DebugFormat("[SynchronousRestObjectRequester]: Invalid XML {0} {1}", requestUrl, typeof(TResponse).ToString()); | 997 | m_log.DebugFormat( |
998 | "[SynchronousRestObjectRequester]: Invalid XML from {0} {1} {2}", | ||
999 | verb, requestUrl, typeof(TResponse).ToString()); | ||
981 | } | 1000 | } |
982 | catch (Exception e) | 1001 | catch (Exception e) |
983 | { | 1002 | { |
984 | m_log.DebugFormat("[SynchronousRestObjectRequester]: Exception on response from {0} {1}", requestUrl, e); | 1003 | m_log.DebugFormat( |
1004 | "[SynchronousRestObjectRequester]: Exception on response from {0} {1}: {2}{3}", | ||
1005 | verb, requestUrl, e.Message, e.StackTrace); | ||
985 | } | 1006 | } |
986 | 1007 | ||
987 | return deserial; | 1008 | return deserial; |
988 | } | 1009 | } |
989 | } | 1010 | } |
990 | } | 1011 | } \ No newline at end of file |
diff --git a/OpenSim/Region/Framework/Scenes/Scene.cs b/OpenSim/Region/Framework/Scenes/Scene.cs index 7a2b2ed..a46c827 100644 --- a/OpenSim/Region/Framework/Scenes/Scene.cs +++ b/OpenSim/Region/Framework/Scenes/Scene.cs | |||
@@ -3217,8 +3217,8 @@ namespace OpenSim.Region.Framework.Scenes | |||
3217 | try | 3217 | try |
3218 | { | 3218 | { |
3219 | m_log.DebugFormat( | 3219 | m_log.DebugFormat( |
3220 | "[SCENE]: Removing {0} agent {1} from region {2}", | 3220 | "[SCENE]: Removing {0} agent {1} {2} from region {2}", |
3221 | (isChildAgent ? "child" : "root"), agentID, RegionInfo.RegionName); | 3221 | (isChildAgent ? "child" : "root"), avatar.Name, agentID, RegionInfo.RegionName); |
3222 | 3222 | ||
3223 | m_sceneGraph.removeUserCount(!isChildAgent); | 3223 | m_sceneGraph.removeUserCount(!isChildAgent); |
3224 | 3224 | ||
diff --git a/OpenSim/Region/Framework/Scenes/ScenePresence.cs b/OpenSim/Region/Framework/Scenes/ScenePresence.cs index e8178ce..7a94215 100644 --- a/OpenSim/Region/Framework/Scenes/ScenePresence.cs +++ b/OpenSim/Region/Framework/Scenes/ScenePresence.cs | |||
@@ -1690,9 +1690,9 @@ namespace OpenSim.Region.Framework.Scenes | |||
1690 | if (pos.Z - terrainHeight < 0.2) | 1690 | if (pos.Z - terrainHeight < 0.2) |
1691 | pos.Z = terrainHeight; | 1691 | pos.Z = terrainHeight; |
1692 | 1692 | ||
1693 | m_log.DebugFormat( | 1693 | // m_log.DebugFormat( |
1694 | "[SCENE PRESENCE]: Avatar {0} set move to target {1} (terrain height {2}) in {3}", | 1694 | // "[SCENE PRESENCE]: Avatar {0} set move to target {1} (terrain height {2}) in {3}", |
1695 | Name, pos, terrainHeight, m_scene.RegionInfo.RegionName); | 1695 | // Name, pos, terrainHeight, m_scene.RegionInfo.RegionName); |
1696 | 1696 | ||
1697 | if (noFly) | 1697 | if (noFly) |
1698 | Flying = false; | 1698 | Flying = false; |