sifue/n-coin-bot

bot に対して送金出来てしまう

Opened this issue · 1 comments

概要

基本的には死蔵されてしまうのではないか、と感じます。
n-coin-bot自身は現在じゃんけんと相性診断にてコイン残高を利用するため robot.adapter.self.id に対しては開放するか、あるいは nc+1 のリアクション時のみチェックするか、その両方で良いと思っています。

解決

単に bot であるかは User インスタンスの is_bot フラグ ("user type | Slack") を確認するのみです。sendCoin 関数で対象ユーザーについても User インスタンスで受け取るようにすると良いのではないかと思います (次のように: https://github.com/yukarium/n-coin-bot/commit/072bdbc88ac2a50aa4043bd0413164b859f4d23d)

sifue commented

現在、ボットアカウントからは最初100NCを剥奪しています。またジャンケン機能などボットの残高を使うものもあるため、当分はボットにも送れても良いんじゃないかと思ってます。
なおボットは全て管理者に設定してますので、管理者に送金できないようにする、でも対応できますね。