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

2024リニューアル:ImageCDN-Bun #389

Merged
merged 71 commits into from
Aug 10, 2024
Merged

Conversation

gentksb
Copy link
Owner

@gentksb gentksb commented Jun 22, 2024

ねらい

  • ビルド時間の大半がレスポンシブ画像のレンダリングに費やされているため、SSR化してリクエスト時にCloudflare Image Resizingを使った画像変換に置き換えてアクセス時に画像を生成+キャッシュする方針に切り替える SSR化はとりやめ、unpicによる画像CDNサービス利用のみを行う
  • Bunランタイムの利用によってビルド・レンダリングの高速化を狙う(コスト削減およびUX・DX向上)
  • その他、Astro組み込みの機能によってカスタムコードを最小限に抑える

考慮事項

個別の記事ページのみ、SSRする。それ以外のページ(特に一覧系)はメモリに全記事を読み込むと表示遅延やWorkersのリソース制限に引っかかるなどのデメリットが大きいため、プリレンダリングしておく。
この方針でもWorkerの無料プラン制限を超えるため、SSG前提にする

Cloudflareの機能を活用する。少なくともPartyRockをCloudflare Zaraz に置き換えることにチャレンジ。ロギングもbaselimeを利用する

テストに加えエラーログとSlack通知を実装する。

ToDo

  • Bunランタイムの適用
  • LinkBox コンポーネントの分割
  • unpicによるimgix配信テスト->被採用。Cf image resizingでやる
  • OGP画像生成のフォントフェッチ
  • OGP APIとPAAPI APIにCORSを付与
  • アフィリエイトリンクにブランドカラーを付与する
  • 引用のためのシンプルリンクコンポーネントを作成する
  • エラーログ通知機能実装
  • PageFindの実装
  • PartyRockをZarazに置き換える
  • ExcerptをContainer APIで実装しなおす
  • Twitter, instagramのリンクをZarazによる実装への置き換えを試す(SSGでも実行できるか未知数) 使い物にならないので却下
  • タグページの作成 SSGだとタグを型で縛る必要があるので別途実装

お掃除

  • テスト用ドメインの設定削除
  • Zarazで純正Twitterスクリプトの読み込み設定をする
  • Zarazで純正ebayスクリプトの読み込み設定をする
  • VCタグを直接埋め込みGTMを無効化する

Copy link

cloudflare-workers-and-pages bot commented Jun 22, 2024

Deploying blog with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3a6bcc7
Status:⚡️  Build in progress...

View logs

@gentksb gentksb force-pushed the migration-to-ssr-cfimage-biome branch from 5ec09ed to ba553a6 Compare June 22, 2024 12:50
@gentksb gentksb force-pushed the migration-to-ssr-cfimage-biome branch from 00ef665 to a61631e Compare June 23, 2024 10:44
@gentksb
Copy link
Owner Author

gentksb commented Jul 25, 2024

各所にtext-wrapのCSS属性を追加していく
https://yuheiy.com/2024-07-22-text-wrap-in-japanese

@gentksb
Copy link
Owner Author

gentksb commented Aug 8, 2024

バリューコマースタグだけZarazで動作しない…

@gentksb gentksb merged commit 19682cf into master Aug 10, 2024
3 of 4 checks passed
@gentksb gentksb deleted the migration-to-ssr-cfimage-biome branch August 10, 2024 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant