Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

webp画像が存在する場合に、.jpgや.pngでアクセスが来てもwebpを返す様に変更 #4900

Merged
merged 1 commit into from
Mar 9, 2021

Conversation

tao-s
Copy link
Contributor

@tao-s tao-s commented Feb 4, 2021

概要(Overview・Refs Issue)

Web Core Vitalsの改善のためにwebp対応した場合の設定を.htaccessに追加

ref: #4896 #4807

方針(Policy)

aaa.jpgがあったらaaa.jpg.webpが生成される想定での設定です。

実装に関する補足(Appendix)

ファイルアップロード時か、バッチでやるかは不明ですが、自動でwebp画像が生成される想定です

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか

@chihiro-adachi chihiro-adachi added this to the 4.1 milestone Feb 9, 2021
@chihiro-adachi chihiro-adachi added the enhancement 機能追加 label Feb 9, 2021
@okazy okazy self-assigned this Mar 2, 2021
@okazy
Copy link
Contributor

okazy commented Mar 4, 2021

画像変換は以下のスクリプトで行いました。

#!/bin/sh

images=`find . -type f -name '*.jpg' -or -name '*.jpeg' -or -name '*.png'`

for image in $images;
do
    cwebp $image -o $image".webp" >/dev/null 2>&1
done

Chrome/webpファイルあり

問題なくwebpファイルが返りました。

image

Chrome/webpファイルなし

問題なくpngファイルが返りました。

image

Safari

問題なくpngファイルが返りました。

image

lighthouse のスコアに有意な差は確認できませんでした。

@okazy
Copy link
Contributor

okazy commented Mar 5, 2021

@tao-s
確認遅くなりました 🙏
動作問題ありませんでした。

4.1の新機能追加については 4.1-feature ブランチにまとめたいと思っています。
お手数なのですが、 4.1-feature ブランチの方にプルリクを出しなおしていただけないでしょうか?
4.1-feature ブランチは 4.0 ブランチベースとなるので、 cherry-pick をしていただく流れになるかと思います。

@okazy okazy changed the base branch from 4.1 to 4.1-feature March 9, 2021 04:13
@okazy okazy changed the base branch from 4.1-feature to 4.1 March 9, 2021 04:13
@okazy okazy force-pushed the 4.1/webp_htaccess branch from 9266f5f to b71f248 Compare March 9, 2021 04:25
@okazy okazy changed the base branch from 4.1 to 4.1-feature March 9, 2021 04:26
@okazy
Copy link
Contributor

okazy commented Mar 9, 2021

4.1-feature ブランチ派生に切り替えました。

hub checkout https://github.com/EC-CUBE/ec-cube/pull/4900
git reset --hard c8e55c5bc4b0e20543d61e8d76c5a36784bab565
git cherry-pick 9266f5f14368f143481561aafefdfd542d687bb1
hub push --force

@tao-s
Copy link
Contributor Author

tao-s commented Mar 9, 2021

@okazy ありがとうございます。
別で画像変換+サムネイル生成もPR出しときます

@okazy okazy merged commit 212ea0e into EC-CUBE:4.1-feature Mar 9, 2021
@okazy
Copy link
Contributor

okazy commented Mar 9, 2021

@tao-s
ありがとうございます!取り込みました。
画像変換・サムネイル生成もよろしくお願いします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants