capsctrldays

2010-12-04 (土) [長年日記]

[tDiary] svn up -r 3663 あたりでCromeで見れなくなる

3663から3665あたりで問題が起きている気がするので、 上げたり下げたりしたけど、よくわからん。

生成されるヘッダに違いはない。何だろう。Safariでは見れる。

追記

Cromeでソース見たら</html>が欠けてるなあ。他で見たら問題はない。

追記

bash-2.05$ svn diff -r 3664:HEAD
Index: dispatcher.rb
===================================================================
--- dispatcher.rb	(revision 3664)
+++ dispatcher.rb	(revision 3666)
@@ -234,6 +234,7 @@
 			begin
 				$stdout = raw_result; $stderr = dummy_stderr
 				result = @target.run( cgi )
+				result.headers.reject!{|k,v| k.to_s.downcase == "status" }
 				result.to_a
 			ensure
 				$stdout = stdout_orig

これっぽいけど、何が悪いのかは分からない。

追記

かくたにさんによって修正されました。

本日のツッコミ(全2件) [ツッコミを入れる]
1 ただただし (2010-12-05 (日) 09:47)

このパッチを当てていいものかどうか、実はあんまり自信がなかったんだよね……。Statusを出すべき環境と出してはいけない環境があるような気がする。実行環境を含めてdevelに投げてみてもらえます?

2 かくたに (2010-12-05 (日) 23:00)

tdiary-develにも投稿しましたが、これは注意漏れでした https://gist.github.com/729096