【2/29】ヤマダゲストログイン機能 #3367

概要

ヤマダゲストログイン機能

対応

ゲストログイン時はguest=1をcookie登録
ゲストから正規ユーザになった後、sessionが切れた際に、
・cookieからguest=1を削除
・user_platformのpuidを新sapkeyに更新
・更新情報をログテーブルに追加

テーブル

CREATE TABLE `log_yamada_puid_transition` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
  `user_no` int(11) unsigned NOT NULL,
  `puid` char(255) DEFAULT NULL,
  `puid_guest` char(255) DEFAULT NULL,
  `reg_date` datetime NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

#カラム追加
ALTER TABLE `user_platform` ADD `guest_flg` tinyint(4) unsigned DEFAULT '0';
UPDATE user_platform SET guest_flg = 0 WHERE ISNULL(guest_flg);

テスト方法、確認した内容

APIを使用しない確認方法

sapkey切り替え

1. header情報にsapkeyを追加

2. ソースコードの更新
SubFunc_yamada.inc/S_yamada_Login()で以下コメントアウト
・L11: return S_yamada_LoginWithoutPlatform();
以下のmemcacheを空にする
・'user_no_'.$pt

YamadaWithoutAPI/checkToken()で以下調整
・sapkey：新しいsapkeyを設定（任意の文字列）
・guest：確認内容に応じてtrue/falseに変更
・sapkeyOld：headerに設定してあるsapkeyを設定

3. urlパラメータにtokenを追加してアクセス

切り替え後確認

以下のテーブル情報を確認
・user_platformのpuidを新しいsapkeyへ更新
・log_yamada_puid_transitionにログ作成

header情報のsapkeyを新しいsapkeyに更新
以下のmemcacheを空にする
・'user_no_'.$pt
再度アクセスしてプレイできるか

TODO

• [x] 開発確認

@kouho
レビューお願いします。

@wada
S_yamada_LogGuestTransition関数が見つかりませんでした。
S_yamada_LogPuidTransition関数どちらかに名前を合わせる必要があると思います。

@wada
log_yamada_puid_transitionは1ユーザー最大1レコード入る想定ならuser_noはUNIQUEキーを入れても良いのではと思いました

@kouho
> S_yamada_LogGuestTransition関数が見つかりませんでした。

修正しました。

@kouho
> log_yamada_puid_transitionは1ユーザー最大1レコード入る想定ならuser_noはUNIQUEキーを入れても良いのではと思いました

今のところ想定していないので、対応予定はありません。

@wada
修正確認しました

@wada
最低限のログイン処理はできていそうですが、ゲストログインのユーザーかどうかをcookieだけでなくゲーム側のDBにも残しておけませんか？

@wada
log_yamada_puid_transitionのデータの残し方はpuid_guestにゲストログインのsapkeyが入る想定ではなく、puid側にゲストログイン時のsapkeyが入るのでしょうか？
フィールド名からデータの入り方が逆かと思いました

ゲーム側対応必要箇所

・認証周り
　和田さんがこのプルリクで実装済み

・毎日のヤマダポイント配布
　ゲストユーザーの場合は受取をできないように

ゲーム側で対応不要

・課金部分
　現状のログイン画面へのリンクで問題なし

@kouho
最低限のログイン機能のみ実装する認識なのですが、以下ファイルの情報以外で@kouho さんが認識していることがあれば教えてください。

https://trello.com/1/cards/65c5d9e5ebd936174433f3e4/attachments/65c5e229d6c7471da95139f9/download/%E3%82%B2%E3%82%B9%E3%83%88%E3%83%AD%E3%82%B0%E3%82%A4%E3%83%B3%E8%AD%B0%E4%BA%8B%E9%8C%B2.txt

@wada
元々和田さんの認識のとおりでしたが、他の導線周りが残ることについて確認できておりませんでした。
対応が足りているかどうかというところの確認を2/28 13:00からのYGMTGでも確認いたします。

@wada
ログイン機能以外では以下の表示周りの対応が必要でした。

①ゲストログインの場合はデイリーミッションのヤマダポイントの受取のボタンをグレーアウトして押せないように

②ヤマダポイントキャンペーンの告知にゲストログインの場合はポイントを獲得できない旨の表示を追加

@kouho
把握できている情報量が違うよう（この結論に至った過程がわからない）なので、
以下の対応が必要な理由を、原因と合わせて教えてください。

> ログイン機能以外では以下の表示周りの対応が必要でした。
> ①ゲストログインの場合はデイリーミッションのヤマダポイントの受取のボタンをグレーアウトして押せないように
> ②ヤマダポイントキャンペーンの告知にゲストログインの場合はポイントを獲得できない旨の表示を追加

@wada
①について
ゲストログインではヤマダポイントやヤマダコインは獲得できない仕様であると聞いております。そのため、
ゲストログイン状態でパイレーツパスのデイリーミッションのヤマダポイントを受け取った際に受け取れた表示は出て実際に受け取れていないから問題であると判断してヤマダポイントの受取のボタンをグレーアウトする必要があるかと思いました。

②について
ゲストログインではヤマダポイントやヤマダコインは獲得できない仕様であると聞いておりますので、ゲストログインの場合は獲得できずログインすると獲得できる旨の表示が必要であると考えています。

ゲストログインでヤマダポイントやヤマダコインが獲得できない旨の記載箇所
↓
APi資料内に赤文字の※にゲスㇳログインの場合についての記載が追加されております。
http://gpf-doc.ymdunisys.jp/?page_id=520
http://gpf-doc.ymdunisys.jp/?page_id=529

@kouho
共有ありがとうございます。
一度、実装すべき内容を整理します。

ありがとうございます。

@kouho
対応内容をまとめました、過不足ないか確認お願いします。
https://docs.google.com/spreadsheets/d/1FFVk4Uj3r_pMEA8iifHdGJgJ9r2_toRd-lEIObmbpCY/edit#gid=0

@wada
対応内容のまとめありがとうございます。確認いたしました。和田さんのまとめていただいた内容で過不足はなさそうです。

@kouho
以下対応しました。

user_platformテーブルにゲストカラム追加
ゲストユーザのコイン減算とポイント加算APIの利用制限
パイレーツパスのヤマダPクエストの受取を制限
・モーダルは対応に時間がかかるため不採用
ゲスト状態の不整合を回避するため、島ページでのゲスト状態更新を追加

PresentBoxManagerのtodoは、呼び出し前にチェックが入るので対応保留

@wada
対応ありがとうございます。対応確認いたしました。