search

dounokouno / TransmitMail

Copyright表示不要のPHP製汎用メールフォームシステム
http://etc.dounokouno.com/TransmitMail/
MIT License
142 stars 48 forks source link

checkDenyHostの$this->serversを$this->serverに修正 #29

Closed khsk closed 8 years ago

khsk commented 8 years ago

deny_hostconfigを設定し、checkDenyHostを呼び出すと以下のエラーがでます。

PHP Notice:  Undefined property: TransmitMail::$servers in /TransmitMail-master/lib/TransmitMail.php on line 352
PHP Notice:  Undefined property: TransmitMail::$servers in /TransmitMail-master/lib/TransmitMail.php on line 353

原因は、 getRequestで$this->serverに代入しているのに、 checkDenyHostでは$this->serversを参照している ことだと思われます。

dounokouno commented 8 years ago

PR ありがとうございます！確認します！

dounokouno commented 8 years ago

ご指摘の通りでしたので、マージさせていただきました。

（また、別の箇所（ https://github.com/dounokouno/TransmitMail/commit/feede0a17ad87bf01afdc536887be321d379af07 ）にもコードの間違いがあり、アクセス拒否設定が正しく動作しないことを確認しましたので、マージ後に修正をしました。）