DodontoF icon indicating copy to clipboard operation
DodontoF copied to clipboard

changePlayRoomコマンドのパスワードの処理を変更

Open op3kitt opened this issue 7 years ago • 1 comments

ユーザーがログイン中のプレイルームのパスワードが変更された場合、 パスワードの入力欄の値をサーバーから通知されたcrypt済みの値に変更します。

プレイルームの情報をchangePlayRoomコマンドで変更した時に、 新しいパスワードとcrypt済みのパスワードの文字列が一致した場合、 ユーザーはパスワードの内容を変更していないのでプレイルームのパスワードを変更しません。

op3kitt avatar Dec 06 '17 10:12 op3kitt

この変更の目的ですが、以下のような流れで複数のユーザーが操作を行った際に不具合が発生します。

・不具合の発生手順 1.ユーザーAがパスワード付きのプレイルームにログインする。 この時のプレイルームのパスワードはXとする。

2.ユーザーBがパスワード付きの同プレイルームにログインする。 この時のプレイルームのパスワードはX。

3.ユーザーAがプレイルームのパスワードを変更する。 この時の新しいプレイルームのパスワードをYとする。

4.ユーザーBがプレイルームのタイトルを変更する目的でプレイルーム情報を変更する。 この時、プレイルームのパスワードがXに変更される。

・問題点 プレイルームのタイトルのみを変更したい場合でもプレイルームのパスワードが変更される。

・解決案 プレイルームのパスワード欄を変更しなければパスワードは変更しない。 (本PRの内容です)

ご確認よろしくお願いします。

op3kitt avatar Dec 06 '17 10:12 op3kitt