search

iot-social-network-link / meetalk

1 stars 0 forks source link

room id変えても同じルームでトーク出来てしまう疑惑 #73

Closed ykubot closed 8 years ago

ykubot commented 8 years ago

共通サーバでテストしてみてroom idが違うユーザ同士でトーク出来てしまうように見えたので要調査

seiya-n21 commented 8 years ago

まじで？ 前回テストしたページで再確認してみたけど振り分けられてると思う。

・調査内容 以下の２つのroomidを指定したサイトをブラウザで複数立ち上げて確認

・roomに"room1"を指定 https://nsweb01.cloudapp.net/SkyWay-MultiParty-master/examples/multiparty-sample_rm1.html ・roomに"room2"を指定 https://nsweb01.cloudapp.net/SkyWay-MultiParty-master/examples/multiparty-sample_rm2.html

seiya-n21 commented 8 years ago

コード確認した。パラメータが間違ってると思われる

"room_id"→"room"かな ＃修正初心者のため誰か修正おなしゃす。。。 　今から勉強します。。。

・app/assets/javascripts/video_chat.js

//先に、manage_***()の定義が必要
function video_chat_start(s_roomid, s_name, s_gender, uid) {
    multiparty = new MultiParty( {
        "key": "44ed614d-25eb-4a1f-b7a8-a47acd9f7595",
        "reliable": true,
        "room_id": s_roomid, // ここ
        "debug": 2
    });
xkumiyu commented 8 years ago

どのバージョンのコード見てる？developの最新版はroomになってるよ？ https://github.com/iot-social-network-link/meetalk/blob/develop/app/assets/javascripts/video_chat.js#L82-91

ykubot commented 8 years ago

room id違うのに同じルームに接続された問題、もしかしてsessionStorageに保存された前回のPeerIDを使って接続されたからかもしれない。 https://github.com/nttcom/SkyWay-MultiParty/issues/5

skyway関連でわからないことあったら技開のリポジトリにissue立てよう笑

matsu0228 commented 8 years ago

くぼっち、これありえるな。

まずは下を参考に、コンソールログに、sessionStorage値を表示するように修正するわ！仮説検証しよ。 http://labs.html5go.info/webstorage/sessionstorage

その仮説正しければ、毎回room入室前に、sessionStorage値を削除する機能いれればOKやね

matsu0228 commented 8 years ago

下記も合わせて、本番サーバで試験する。 複数人で接続した場合のroom管理 https://github.com/iot-social-network-link/meetalk/issues/54

ykubot commented 8 years ago

session storageのクリアと、room idの引き渡しを修正して解決したので、一旦closeします。