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

boot.jsの調整 #13536

Closed
wants to merge 4 commits into from
Closed

boot.jsの調整 #13536

wants to merge 4 commits into from

Conversation

tamaina
Copy link
Contributor

@tamaina tamaina commented Mar 6, 2024

What

  • misskey_loaderにimportのPromiseをセットしてブラウザにimportを忘れられないようにした
  • boot実行関数は同期に
  • meta#misskey_metaのdata-generated-atを読んでhtmlの内容が古い場合エラーを出すように
  • DOMContentLoaded前にrenderErrorが発行されてもエラーが描画されるように
  • renderErrorで、Errorインスタンスの場合はname, message, stackを描画するように
  • localeの初期化はmeta#misskey_metaを使うように

Why

あわよくば #12356 でエラーが描画されるかも?

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@tamaina tamaina marked this pull request as ready for review March 6, 2024 09:52
@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Mar 6, 2024
Copy link
Contributor

github-actions bot commented Mar 6, 2024

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

Copy link

codecov bot commented Mar 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 40.22%. Comparing base (61eec93) to head (b34df24).
Report is 122 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13536       +/-   ##
============================================
- Coverage    66.21%   40.22%   -26.00%     
============================================
  Files          990     1521      +531     
  Lines       117137   188551    +71414     
  Branches      4489     2631     -1858     
============================================
- Hits         77568    75851     -1717     
- Misses       39538   112159    +72621     
- Partials        31      541      +510     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tamaina tamaina changed the title perf: boot.jsの調整 boot.jsの調整 Mar 6, 2024
@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Mar 7, 2024

meta#misskey_metaのdata-generated-atを読んでhtmlの内容が古い場合エラーを出すように

CloudflareでHTMLがキャッシュがされるようになっていたりしたら無限ロードに入ってまずいかも

また、もしCloudflareでキャッシュすること前提にmisskey_metaを消していたらデータ取れなくなるかも(フロントエンド側では、misskey_metaがなかった場合は/api/metaにフォールバックするようになっている)

@tamaina
Copy link
Contributor Author

tamaina commented Jul 17, 2024

conflict resolved

@tamaina
Copy link
Contributor Author

tamaina commented Jul 17, 2024

CloudflareでHTMLがキャッシュがされるようになっていたりしたら無限ロードに入ってまずいかも

また、もしCloudflareでキャッシュすること前提にmisskey_metaを消していたらデータ取れなくなるかも(フロントエンド側では、misskey_metaがなかった場合は/api/metaにフォールバックするようになっている)

古いHTMLが配信されること自体、古いバージョンのスクリプトを読み込むことで不整合が発生するのでNGだと思う

@tamaina
Copy link
Contributor Author

tamaina commented Jul 17, 2024

そもそもcache-controlを尊重しないCDNやリバースプロキシってあるのかしら(少なくともnginxやCloudflareはちゃんとしている)

@tai-cha
Copy link
Contributor

tai-cha commented Jul 17, 2024

CFのキャッシュってパージできるはず

@tamaina
Copy link
Contributor Author

tamaina commented Jul 20, 2024

機運が成熟していないのでclose

@tamaina tamaina closed this Jul 20, 2024
@kakkokari-gtyih kakkokari-gtyih deleted the boot-mod branch July 20, 2024 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants