SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

Fix #312. #313

Closed krtx closed 9 years ago

skymountain commented 9 years ago

これ既に .htaccess があったら何もしない方が安全な気がします.

krtx commented 9 years ago

既に .htaccess がある場合でも、設定が変わっている可能性もあるので、書き換える必要はあると思います。

skymountain commented 9 years ago

むしろアプリケーション固有の設定を書いていたのにデフォルトに上書きすることの方が問題ありそうですが...警告とか出せば十分じゃないですか?

krtx commented 9 years ago

デフォルトとは何でしょうか?

krtx commented 9 years ago

警告はあったほうが良いですね。

skymountain commented 9 years ago
AuthType Digest
AuthUserFile "#{conf[:master, :authn, :htdigest]}"
AuthName "#{conf[:master, :authn, :realm]}"

のことです.

krtx commented 9 years ago

それにプラスで何か書くようなケースってどんな場合があるでしょうか(想定してなかった)

skymountain commented 9 years ago

具体的なケースはわかりませんが,可能性はあると思います.で,もしそういう場合があると上書きすると取り返しが付かなくなるので,できるだけ fail safe な側をとりましょうという提案です. config を変更して .htaccess を上書きする必要がある程度発生すると想定しているのであれば(個人的にはこれがどの程度起こるのかは少々疑問ですが)

ことを奨めます.

krtx commented 9 years ago

プロンプトを作って安全側に傾けました。

開発や運用の設定の段階では何回か rake htaccess をすることがあるので、この PR をした次第です。

skymountain commented 9 years ago

十分だと思います.ありがとうございます :bow:

krtx commented 9 years ago

:bow::bow::bow: