apiのレスポンスのjson_decodeにエラー処理を追加 #1560

概要

curl_execでヤマダAPIから返ってきたデータをjson_decodeする際にエラーになったけどログに残せないケースがありました。(本番ではSyntax errorとしてerror2.txtに残る。)
exec_postのjson_decodeに例外処理を追加してjson_decodeでエラーをキャッチした際はログに残してエラーページを表示させてます。

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

exec_postでjsonDecodeの直前に$bodyにエラーとなるデータ(例:"test")を入れてexec_postが使われている処理を通す
log/yamada_api_log.txtにエラーが残ってエラー画面が表示されればOK。

課金実行のAPIを確認するときはexec_post関数の以下の箇所を修正したら確認できます。
balance APIでエラーが起きると課金実行までいけないので以下のようにifで省いてください。

curl_close($req);

↓

curl_close($req);
if($end_point != $this->api_host . "/api/balance"){
　$body = "test";
}

• ガチャ実行前残額表示
• ガチャ実行
• デイリークエスト達成のポイント付与

TODO

懸念点
apiへの送信に成功したけど受信したデータのjsonDecodeに失敗した場合2重で処理が行われる可能性

@wada @iwamoto
確認お願いいたします。

TODOに書いていますが、apiへの送信に成功してヤマダ側では処理が通ったけど受信したデータのjson_decodeに失敗したというケースに対応できず何かいい方法はないでしょうか？

いざ問題が発生した際にbodyの中身をyamada_api_error.txtに吐き出して確認しようと思いますが、
例えばaddPoint APIで上記の問題が起きた場合、ポイント付与が2回以上行われてしまう可能性があります。

@kouho
確認okです

@kouho 確認しました。