【8/23】リクエストヘッダに応じてyamadaAPIを使用しない #1032

概要

PFで問題がある時にyamadaAPIを使用すると、エラーでアプリをプレイできないので、
リクエストヘッダにsapkey設定時、yamadaAPIを使用しない
所持コインは10万を基準にキャッシュで管理
（12時間保持して、時間切れで10万にリセット）

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

リクエストヘッダに以下をModHeaderなどで設定

sapkey: {使用したいsapkey}

他人のsapkeyを設定すると、そのユーザでプレイできます
登録していないsapkey（適当な文字列でも可）を設定すると、ユーザ登録されます
チュートリアルやデバッグ用ユーザの追加に利用してください

TODO

apiの戻り値は、使用していないものはnullかダミーデータを返しています
必要に応じてダミーデータ追加します

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

実際にpfで問題が出ないと正しい効果は確認できないですが、
yamadaAPIを使用しないことで、デバッグで以下のメリットがあります。
・所持コイン数を改竄
・新規ユーザの登録が簡単
・他のユーザへの成り代わり

@iwamoto
S_yamada_GetUserDataBySapKeyの引数について、
使用している箇所は修正、使用していない箇所は削除しました。

@wada
以下確認しました。動作は問題なかったです。

• HTTP_SAPKEYにsapkeyを格納してのユーザーなりすまし
• HTTP_SAPKEYにsapkeyを格納しての新規ユーザー登録
• 所持コイン数の改鋳と使用
• YamadaApiでエラー時に動作

exec_get, exec_postの関数で以下を強制的にreturnさせてゲームを動かしてみました

		return [
			'code' => 404,
			'info' => [],
			'data' => [],
		];

確認時に リクエストヘッダに以下をModHeaderなどで設定
こちらに手間取りましてMod Headerやその他のGoogle Chromeの拡張プラグインを使ってもヘッダーの書き換えができませんでした。最終的に/etc/nginx/conf.d/jrye-web.conf を直接書き換えることでヘッダーの書き換えに成功しました。
環境の違い等に起因するものかもしれませんが一応共有です。

@wada
修正箇所確認しました大丈夫ですー

下記一通り確認しましたが、問題ないように思います。
・所持コイン数を改竄
・新規ユーザの登録が簡単
・他のユーザへの成り代わり

@wada
実処理では通っていないので、問題はないかもしれませんが…
checkTokenの引数($token)が使用されてません。
$token = $pt の間違いかと思われます。

@wada
S_yamada_GetUserDataBySapKeyの引数間違いになります。

修正前は
`「他のユーザへの成り代わり」を行った際に
自分の島へ移動してしまうと、user_info.nicknameが "名無し" に上書きされる` 状態になっていました