Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IRCプロトコルにあわせたメッセージの自動切断 #24

Closed
koi-chan opened this issue Feb 25, 2015 · 6 comments
Closed

IRCプロトコルにあわせたメッセージの自動切断 #24

koi-chan opened this issue Feb 25, 2015 · 6 comments

Comments

@koi-chan
Copy link
Member

cinch の、IRCコマンドが長すぎた場合にちょうど良い長さで出力を切る機能は、2バイト文字だとうまく動作しない可能性がある。
以下の2つの動作を確認した。

  • IRCサーバー側から Request Too Long エラーによって強制切断される。
  • 途中で切断しても文字の途中で切断されて、最後の文字が化ける。

cinch自体の改良を行なうか、RGRB側でメッセージの自動調整機能を実装したい。

@ochaochaocha3
Copy link
Member

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

@ochaochaocha3
Copy link
Member

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

@koi-chan
Copy link
Member Author

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

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のコードから、空白の部分できるようになっていることが分かりますが、日本語の場合は単語の間に空白文字列が挟まれることは基本的にないため、空白文字列が現れる前の何文字かが失われたようです。

@ochaochaocha3
Copy link
Member

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

  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! とも文字のインデックスを指定する関数だから)。文字ごとに累積バイト数を求め、最大バイト数以下の最大値になるところで切ってあげれば、文字のバイト数に関係なく適切に分けることができます。

@ochaochaocha3
Copy link
Member

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

@ochaochaocha3
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants