IRCプロトコルにあわせたメッセージの自動切断 about rgrb HOT 6 CLOSED

これは Cinch のバグと言っていいんじゃないかと思います。なんとか直して PR として送れれば…

from rgrb.

処理は https://github.com/cinchrb/cinch/blob/9b66b216ad74471e059974c3c5fc121e2c61d7c2/lib/cinch/target.rb#L46-L59 のあたりで行われているようです。

from rgrb.

もう一つ異常動作がありました。

16:16:45 (koi-chan) .rg asw00
16:16:45 (rgrb_cinch) rg[koi-chan]<asw00>: 殺人事件。主人公は被害者(またはその家族など)に同情し、事件の解決に乗り出す。被害者は信心深い庭師。加害者は一度死んだはずの。主人公と敵対している人物。殺害の動機はなんらかの野望(権力を握る、財産を得る)ために邪魔なため。ダークネスに命じて殺害。事件の真相を突き止め、犯人[CUT]
16:16:45 (rgrb_cinch) ... ですわ☆

IRCでは以下のような応答がありましたが、コンソールにIRCへ流す文字列を流すようにしたところ、以下のような出力をしていることが分かりました。

"rg[koi-chan]<asw00>: 殺人事件。主人公は被害者(またはその家族など)に同情し、事件の解決に乗り出す。被害者は信心深い庭師。加害者は一度死んだはずの。主人公と敵対している人物。殺害の動機はなんらかの野望(権力を握る、財産を得る)ために邪魔なため。ダークネスに命じて殺害。事件の真相を突き止め、犯人を捕らえれば解決。 ですわ☆"

上で提示されたcinchのコードから、空白の部分できるようになっていることが分かりますが、日本語の場合は単語の間に空白文字列が挟まれることは基本的にないため、空白文字列が現れる前の何文字かが失われたようです。

from rgrb.

日本語で空白が入らないことは関係がないようです。原因は

1. https://github.com/cinchrb/cinch/blob/9b66b216ad74471e059974c3c5fc121e2c61d7c2/lib/cinch/target.rb#L49 （最大バイト数以下で最後にスペースがある場所を求める）
2. https://github.com/cinchrb/cinch/blob/9b66b216ad74471e059974c3c5fc121e2c61d7c2/lib/cinch/target.rb#L51 （求めたバイト数で行を切断する）

の 2 つで、バイト数で計算すべきところを文字数で計算していることです（String#rindex、String#slice! とも文字のインデックスを指定する関数だから）。文字ごとに累積バイト数を求め、最大バイト数以下の最大値になるところで切ってあげれば、文字のバイト数に関係なく適切に分けることができます。

from rgrb.

Cinch に PR を送っています (cinchrb/cinch#189)。

from rgrb.

Dominik Honnef さん (@dominikh) に直していただき、無事マージされました。Cinch の次のバージョンから正常に切断されるようになると思います。

from rgrb.