チェンジセット 1679 (default)

日時:

2024/08/30 22:12:30 (5週前)

更新者:

hizuya@…

ログメッセージ:

• Web レスポンスログに記録される CntentType の charset が HTTP クライアントに返した値とは異なる可能性があったのを修正。

場所:

framework/trunk

ファイル:

• WebLibrary/Properties/Resources.designer.cs (更新) (1個の差分)
• WebLibrary/Properties/Resources.ja.resx (更新) (1個の差分)
• WebLibrary/Properties/Resources.resx (更新) (1個の差分)
• WebLibrary/Sources/Handlers/HttpHandlerBase.cs (更新) (2個の差分)
• WebLibrary/Sources/Reflections.cs (更新) (1個の差分)
• WebLibrary/Sources/WebDebugging.cs (更新) (13個の差分)
• WebTest/Sources/ReflectionsTest.cs (更新) (2個の差分)

framework/trunk/WebLibrary/Properties/Resources.designer.cs

@@ -1832 +1832 @@
         
         /// <summary>
-        ///   {0}; charset={1} に類似しているローカライズされた文字列を検索します。
-        /// </summary>
-        internal static string Log_Web_WebDebugging_ResponseContentTypeWithEncodingValue {
-            get {
-                return ResourceManager.GetString("Log_Web_WebDebugging_ResponseContentTypeWithEncodingValue", resourceCulture);
-            }
-        }
-        
-        /// <summary>
         ///   Web response sent. に類似しているローカライズされた文字列を検索します。
         /// </summary>

framework/trunk/WebLibrary/Properties/Resources.ja.resx

@@ -363 +363 @@
     <value>レスポンス: ContentType</value>
   </data>
-  <data name="Log_Web_WebDebugging_ResponseContentTypeWithEncodingValue" xml:space="preserve">
-    <value>{0}; charset={1}</value>
-  </data>
   <data name="Log_Web_WebDebugging_ResponseHeader" xml:space="preserve">
     <value>Web レスポンスを送信しました。</value>

framework/trunk/WebLibrary/Properties/Resources.resx

@@ -492 +492 @@
 Path: {0}</value>
   </data>
-  <data name="Log_Web_WebDebugging_ResponseContentTypeWithEncodingValue" xml:space="preserve">
-    <value>{0}; charset={1}</value>
-  </data>
   <data name="Log_Web_WebDebugging_ResponseContentTypeKey" xml:space="preserve">
     <value>Response: ContentType</value>

framework/trunk/WebLibrary/Sources/Handlers/HttpHandlerBase.cs

@@ -597 +597 @@
                     if (!ServiceErrorHandlerModule.Registered)
                     {
-                        WebDebugging.LogResponse(logger, ex);
+                        WebDebugging.LogResponse(logger, context.Response, ex);
                     }
                 }
@@ -619 +619 @@
                     if (!ServiceErrorHandlerModule.Registered)
                     {
-                        WebDebugging.LogResponse(logger, httpUnhandledException);
+                        WebDebugging.LogResponse(logger, context.Response, httpUnhandledException);
                     }
                 }

framework/trunk/WebLibrary/Sources/Reflections.cs

@@ -604 +604 @@
 
         /// <summary>
+        /// <see cref="System.Web.HttpResponse"/> に関係するリフレクションの集合です。
+        /// </summary>
+        internal static class HttpResponse
+        {
+            /// <summary>
+            /// <c>AppendCharSetToContentType</c> メソッドを呼び出すデリゲートです。
+            /// </summary>
+            private static readonly Func<System.Web.HttpResponse, string, string> AppendCharSetToContentTypeDelegate = InitializeAppendCharSetToContentTypeDelegate();
+
+
+            /// <summary>
+            /// 使用出来るかどうかを返します。
+            /// </summary>
+            /// <value>
+            /// 使用出来る場合は <see langword="true"/>。
+            /// それ以外の場合は <see langword="false"/>。
+            /// </value>
+            internal static bool IsValid
+            {
+                get
+                {
+                    return AppendCharSetToContentTypeDelegate != null;
+                }
+            }
+
+
+            /// <summary>
+            /// <c>AppendCharSetToContentType</c> メソッドを呼び出します。
+            /// </summary>
+            /// <param name="response">HTTP レスポンス。</param>
+            /// <param name="contentType">コンテンツタイプ。</param>
+            /// <returns>
+            /// <c>charset</c> を付けたコンテンツタイプ。
+            /// </returns>
+            internal static string AppendCharSetToContentType(System.Web.HttpResponse response, string contentType)
+            {
+                if (AppendCharSetToContentTypeDelegate == null)
+                {
+                    return null;
+                }
+
+                return AppendCharSetToContentTypeDelegate(response, contentType);
+            }
+
+
+            /// <summary>
+            /// <see cref="AppendCharSetToContentTypeDelegate"/> を初期化します。
+            /// </summary>
+            /// <returns>
+            /// <c>AppendCharSetToContentType</c> メソッドを呼び出すデリゲート。
+            /// </returns>
+            private static Func<System.Web.HttpResponse, string, string> InitializeAppendCharSetToContentTypeDelegate()
+            {
+                MethodInfo appendCharSetToContentTypeMethodInfo
+                    = typeof(System.Web.HttpResponse).GetMethod(
+                        "AppendCharSetToContentType",
+                        BindingFlags.Instance | BindingFlags.NonPublic,
+                        null,
+                        new[] { typeof(string) },
+                        null);
+                if (appendCharSetToContentTypeMethodInfo == null)
+                {
+                    return null;
+                }
+
+                return delegate(System.Web.HttpResponse response, string contentType)
+                {
+                    return (string)appendCharSetToContentTypeMethodInfo.Invoke(response, new object[] { contentType });
+                };
+            }
+        }
+
+
+        /// <summary>
         /// <see cref="System.Web.UI.Control"/> に関係するリフレクションの集合です。
         /// </summary>

framework/trunk/WebLibrary/Sources/WebDebugging.cs

@@ -29 +29 @@
     using System.Net;
     using System.Security.Principal;
-    using System.Text;
     using System.Text.RegularExpressions;
     using System.Threading;
@@ -402 +401 @@
         /// ログの出力先。
         /// </param>
+        /// <param name="response">
+        /// HTTP レスポンス。
+        /// </param>
         /// <param name="ex">
         /// HTTP 例外。
         /// </param>
-        internal static void LogResponse(ILogger logger, HttpException ex)
+        internal static void LogResponse(
+            ILogger logger,
+            HttpResponse response,
+            HttpException ex)
         {
             if (logger == null || !logger.InformationEnabled(LoggingTargets.Trace))
@@ -420 +425 @@
                     true);
 
-            AppendResponse(builder, ex);
+            AppendResponse(builder, response, ex);
 
             LogInformation(logger, webLogger, Logging.WebWebDebuggingResponse, builder.ToString());
@@ -466 +471 @@
             if (exception != null)
             {
-                AppendResponse(builder, exception);
+                AppendResponse(builder, response, exception);
             }
             else if (response != null)
@@ -609 +614 @@
             AppendResponse(
                 builder,
+                response,
                 response.Status,
                 response.ContentType,
-                response.ContentEncoding,
                 response.RedirectLocation,
                 message);
@@ -622 +627 @@
         /// ログビルダー。
         /// </param>
+        /// <param name="response">
+        /// HTTP レスポンス。
+        /// </param>
         /// <param name="exception">
         /// 例外。
@@ -627 +635 @@
         private static void AppendResponse(
             MultilineTraceStringBuilder builder,
+            HttpResponse response,
             Exception exception)
         {
@@ -639 +648 @@
             AppendResponse(
                 builder,
+                response,
                 string.Format(
                     CultureInfo.InvariantCulture,
@@ -644 +654 @@
                     statusCode,
                     HttpWorkerRequest.GetStatusDescription(statusCode)),
-                null,
                 null,
                 null,
@@ -661 +670 @@
         /// ログビルダー。
         /// </param>
+        /// <param name="response">
+        /// HTTP レスポンス。
+        /// </param>
         /// <param name="status">
         /// HTTP ステータスライン。
@@ -667 +679 @@
         /// コンテンツタイプ。
         /// </param>
-        /// <param name="contentEncoding">
-        /// コンテンツエンコーディング。
-        /// </param>
         /// <param name="redirectLocation">
         /// リダイレクト先。
@@ -678 +687 @@
         private static void AppendResponse(
             MultilineTraceStringBuilder builder,
+            HttpResponse response,
             string status,
             string contentType,
-            Encoding contentEncoding,
             string redirectLocation,
             string message)
@@ -688 +697 @@
             if (!string.IsNullOrEmpty(contentType))
             {
-                if (contentEncoding != null)
-                {
-                    builder.Append(
-                        Resources.Log_Web_WebDebugging_ResponseContentTypeKey,
-                        Utility.Format(
-                            Resources.Log_Web_WebDebugging_ResponseContentTypeWithEncodingValue,
-                            contentType,
-                            contentEncoding.WebName));
-                }
-                else
-                {
-                    builder.Append(Resources.Log_Web_WebDebugging_ResponseContentTypeKey, contentType);
-                }
+                contentType = Reflections.HttpResponse.AppendCharSetToContentType(response, contentType);
+                builder.Append(Resources.Log_Web_WebDebugging_ResponseContentTypeKey, contentType);
             }
 

framework/trunk/WebTest/Sources/ReflectionsTest.cs

@@ -24 +24 @@
 {
     using System.Diagnostics;
+    using System.IO;
+    using System.Text;
     using System.Web;
     using NUnit.Framework;
@@ -289 +291 @@
         #endregion
 
+        #region HttpResponse
+
+        /// <summary>
+        /// <see cref="Reflections.HttpResponse"/> が使用出来るかのテストです。
+        /// </summary>
+        [Test]
+        public void TestHttpResponseIsValid()
+        {
+            Assert.That(Reflections.HttpResponse.IsValid, Is.True);
+        }
+
+        /// <summary>
+        /// <see cref="Reflections.HttpResponse"/> の <c>AppendCharSetToContentType</c> へアクセスするテストです。
+        /// </summary>
+        [Test]
+        public void TestAppendCharSetToContentTypeFromContentEncoding()
+        {
+            using (StringWriter writer = new StringWriter())
+            {
+                // TODO: new HttpContext(HttpWorkerRequest).Response を使用しないと ContentEncoding から charset が設定されない
+                HttpResponse response = new HttpResponse(writer)
+                {
+                    ContentEncoding = Encoding.UTF8,
+                };
+
+                Assert.That(
+                    Reflections.HttpResponse.AppendCharSetToContentType(response, "text/html"),
+                    Is.EqualTo("text/html"));
+                    ////Is.EqualTo("text/html; charset=utf-8"));
+            }
+        }
+
+        /// <summary>
+        /// <see cref="Reflections.HttpResponse"/> の <c>AppendCharSetToContentType</c> へアクセスするテストです。
+        /// </summary>
+        [Test]
+        public void TestAppendCharSetToContentTypeFromCharset()
+        {
+            using (StringWriter writer = new StringWriter())
+            {
+                HttpResponse response = new HttpResponse(writer)
+                {
+                    Charset = "ascii",
+                };
+
+                Assert.That(
+                    Reflections.HttpResponse.AppendCharSetToContentType(response, "text/html"),
+                    Is.EqualTo("text/html; charset=ascii"));
+            }
+        }
+
+        #endregion
+
         #region Control