PR

【今週中】ヤマダアバター画像をhttpsで取得する #706

wada opened this at 2021-03-30T17:09:44; closed

Description

opened by wada at 2021-03-30T17:09:44; updated 2024-04-15T09:27:46

アプリ内でヤマダアバター画像のみhttpで取得していたため、混在コンテンツと判断される
https://ssl.sakura.ad.jp/column/mixed-content-dl

それを避けるためにhttpsで画像を取得するよう対応
apiのレスポンスデータがhttpなので、apiでデータ取得後urlのhttp部分をhttpsに置換

また、すでにuser_infoにhttpで保存されているため、user_infoからアバター画像(thumbnailUrl)取得時にもhttpsに置換

Comment

#10079 by wada at 2021-03-30T17:20:10; updated 2021-03-30T17:20:10

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

以下のページなどでアバター画像のurlがhttpsになっているか。エラーが発生していないか確認お願いします。
・各種ランキング
・海賊団ページ
・自分の島
・覇戦ページ
・冒険中のユーザエンカウント時
・ガチャのレアカード取得者
・ベル履歴
など

ローカルで確認難しいものは開発マージ後に開発環境でチェックできればと思います。

wada add_label at 2021-03-30T17:20:37: レビュー待ち

Comment

#10083 by kouho at 2021-03-30T18:39:34; updated 2021-03-30T18:45:14

@wada
記載されている中でベル履歴以外確認しました。
ガチャのレアカード取得者がhttpのままになっていました。
メモキャッシュに残っていた場合、httpのままになる場合があるようです。
それ以外は大丈夫かと思います。

また、本件とは直接関わりがないですが共有です。
天下無双のランキング表示を確認の際に本番のuser_infoをローカルに持ってきていました。
user_infoのthumbnailUrlには本番のavatar画像のURLが入っていたためローカル環境から本番のアバターを見てしまっていました。

Comment

#10088 by wada at 2021-03-30T18:52:41; updated 2021-03-30T18:52:41

@kouho
確認ありがとうございます。
レアカードに関しては、キャッシュに入れる際に対応してます。
時間で消える上、他のユーザが取得すれば上書きされるため、すでに入ってるデータに対しては対応入れてません。(なるべく置換処理を走らせなくない)

画像であればいくらでも見て良いと思いますよ。

Comment

#10089 by kouho at 2021-03-30T19:09:49; updated 2021-03-30T19:09:49

@wada
レアカード取得者と画像取得につきましてかしこまりました。

kouho add_label at 2021-03-30T19:09:54: 皇甫 LGTM

Comment

#10091 by iwamoto at 2021-03-30T21:03:18; updated 2021-03-30T21:03:18

@wada
下記2点の確認と修正をお願い致します

1.冒険中のユーザエンカウント時
2.バトル結果画面
こちら2点は htpp のURLでした

1.mission_default.inc で user_infoから取得した値がそのまま使用されています。
$return_data['friend_thumbnailUrl']

2.Battle-Result.inc も 1と同様の問題でした。

ほかは大丈夫かと思います。

Comment

#10103 by wada at 2021-03-31T12:15:37; updated 2021-03-31T12:15:37

@iwamoto
確認ありがとうございます。
冒険中とバトル結果について対応しました。
js内にコロプラ画像を参照している箇所があったのでそちらも対応しておきました。

Comment

#10104 by iwamoto at 2021-03-31T12:30:19; updated 2021-03-31T12:30:19

@wada
修正箇所確認しました。

iwamoto add_label at 2021-03-31T12:30:24: 岩本 LGTM
wada merge at 2021-03-31T15:56:31: 【今週中】ヤマダアバター画像をhttpsで取得する
wada close at 2021-03-31T15:56:31: Close

Commit range

base5fe7867d13a5dc5b88c0b1ec68ca365bcc892aa9
head07d86eaec0e99672550e01d20376317a28356a86
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_avatar_url

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed8 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/SubFunc_mission.inc
modifiedlib/class/platform/yamada/YamadaPlatformService.class.php
modifiedlib/models/dao/user_info.php
modifiedlib/public/Battle-Result.inc
modifiedpublic_html/js/JOLLYROGER_mission.js
modifiedpublic_html/js/jr_mission.js
modifiedpublic_html/template/TeamWar-Result.tpl
Patch body is stored as a separate file to keep this page lightweight.