tdiary serverコマンドにログファイル指定オプションが欲しい

[tdtds]

現在 $stderr 決め打ちだが、ログファイルを指定できた方が良い。

ローテーションとか考え始めると面倒になるけどそれは別途。

[machu]

少し書いてみたのですが、サーバの種類 (thin, webrick, unicorn) ごとに処理を分けないとダメですね。

• tdiary server コマンドは webrick に限定
• それ以外のrackサーバを使いたいときは、 Gemfile.local に書いて bundle exec rackup や bundle exec unicorn などで起動

のほうが良い気がしてきました。

[tdtds]

うーん、なるほど。性能等を確保したい場合にはtdiaryコマンド使うなとなると、tdiaryコマンドはあくまで開発者向けという位置づけになってしまいそう。tdiaryコマンドの扱いを明確にしないといけませんね……。

[machu]

Railsはどうだろうと調べてみました。

• bin/rails serverで起動すると必要なミドルウェアをRails側でセットアップする。
• rackupで起動するとconfig.ruにミドルウェアを書く。

tDiaryでもomniauthのセッションやbase_dirをconfig.ruに書くことから、tdiary serverコマンドは開発やお試し用に割り切るのがよいと思えてきました。

[tdtds]

(ちょっと脱線) omniauth関連のissueでも気になったんだけど、config.ruの扱いも難しいですよね。tDiaryではconfig.ruも配布ファイルだけど、それを書き換えることを前提にするのは(これまでの)tDiaryの流儀からは外れてしまうという。

[machu]

そうなんですよね。CGIでの.htaccessのような性質も持っているので、どう扱うかのポリシーを決めたいですね。

[hsbt]

理想的には tdiary.conf に書いた middlewares を stack に積む機構があるとよさそう

[machu]

設定系の課題は別issueを立てるとして、いったん tdiary serverに話を戻します。

現状（わかっていること、わかったこと）

• tdiary server コマンドでは --cgi (TDiary::Server) と --rack (Rack::Server) による起動をサポート
  • --cgi はwebrickで起動
  • --rack は Rack::Server が対応しているRackサーバ (thin/puma/webrick/mongrel) で起動
• Rack::Server はwebrickのみログファイルの指定が可能
  • https://github.com/rack/rack/blob/master/lib/rack/server.rb#L174 (supporting Rack::Handler次第？)
  • ログファイルの指定はサーバ特化となる (WEBrick::AccessLog::CLFのような記述)
• よって tdiary server は Rack::Server に依存している限り、 webrick 以外のログファイルを指定できない

対応案

1. webrick指定の場合のみ --log オプションを有効にする (cli.rb内で起動サーバを判定できる？)
2. tdiary server コマンドでは --log オプションをサポートしない。ログファイルを指定したいときは、利用者が使っているRackサーバごとに指定する。
   • unicornの場合はunicorn.conf, thinの場合はconfig.yamlや-l logs/thin.log
3. ログの出力をRackサーバに頼らずに、tDiary側で面倒を見る (CommonLoggerを独自でハンドリング)

ちょっと、案3でやってみます。

[machu]

案3のCommonLoggerでやってみましたが、RackサーバではなくRackアプリケーション側でのログ出力になるので、config.ruに指定する形になりますね。

use Rack::CommonLogger,
  File.new('log/access.log', 'a+').tap{|f| f.sync = true }

config.ruは書き換えない前提として、

• tdiary server コマンドは今のままにする
• ログファイルを指定したいときは tdiary server ではなく thin や unicorn 側から起動する
• config.ruの別ファイルへの切り出し (Rackサーバの設定をconfig.ruではなく別ファイル (tdiary.confなど) に書きたい #476) が実現したら、tdiary側でもログを出力できるようにする

という方針でどうでしょうか？

[tdtds]

見落としてました、すみません。

「tdiary server コマンドは今のままにする」ってのはWrbrick専用で--logオプションがある状態という意味ですね? であれば現状はそれで良いと思います。

[tdtds]

これはどういうステータスなんでしたっけ? 「config.ruの別ファイルへの切り出し (#476) が実現したら、tdiary側でもログを出力できるようにする」がまだかな?

[machu]

はい、そうです。

[tdtds]

じゃあ延期しておきます