iot-social-network-link / meetalk

1 stars 0 forks source link

Feature/check session storage #76

Closed matsu0228 closed 8 years ago

matsu0228 commented 8 years ago

ルームidが自分のuser情報と異なる部屋に入っている疑惑(https://github.com/iot-social-network-link/meetalk/issues/73 )への対応。

◆確認内容 ①自分のユーザidを確認(画像では5) ②sessionStorageに格納されているpeerIDを確認 ③sessionStorageクリア後のsessionStorage値を確認 ④3でクリアしたことで、自分のuseidと同じpeerIDで入室できていること(画像では5)  →本対応前では、sessionStorage値②を用いて入室していたみたい meetalk 2

xkumiyu commented 8 years ago
ykubot commented 8 years ago

既に格納されてたらその値を使うとかにできない?

それすると保持されたwindow id使って前回のroomに接続されることになるから、こっちで明示的にクリアした方が良いと思う。

ブラウザの更新したとき、クリアされて別の部屋にはいるとかにならない?

その可能性あるね。ローカルで試してみるけど。 Exitボタン押されたタイミングとか、トーク終了時にクリアするとか? ユーザの挙動に対する処理をディスカッションして決めよう。

logging_debugってcondole.logとは違うの?

フラグで表示を切り替えるために関数作ったってことで合ってる?

matsu0228 commented 8 years ago

sessionStorageをクリアする必要ある?

ある。別の部屋に入ることはない。 前提説明してなくてごめんけど、下記のとおり。

TOP_URL

logging_debug

くぼっちのいうとおり、「DEBUG_FLG」がtrueのときだけcosole.logする関数。 →productionでは、内部情報見せないようにフラグ管理したく。