fix(assets): allow 304 Not Modified status to bypass redirect check while revalidating cached assets
#15935MergedComparing
oliverlynch:main
(c2046e6
) withmain
(ccecb8f
)0%
- Untouched
- 18
No successful run was found on main ( 1130a97) during the generation of this report, so ccecb8f was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.
Benchmarks
18 total
Build: full server site
benchmark/bench/build-server.bench.js::Bench build time-1%
8.7 s8.7 s
Rendering: streaming [false], .astro file
benchmark/bench/render.bench.js::Bench rendering0%
194.2 ms194.8 ms
Build: full static site
benchmark/bench/build-static.bench.js::Bench build time0%
3.8 s3.8 s
Rendering: streaming [false], .mdx file
benchmark/bench/render.bench.js::Bench rendering0%
593 ms591.7 ms
Build: hybrid site (static + server)
benchmark/bench/build-hybrid.bench.js::Bench build time0%
8.9 s8.8 s
Rendering: streaming [true], .md file
benchmark/bench/render.bench.js::Bench rendering0%
1.2 ms1.2 ms
Rendering: streaming [true], .astro file
benchmark/bench/render.bench.js::Bench rendering0%
217.9 ms217.6 ms
many-components (markHTMLString, isHTMLString, validateProps)
benchmark/bench/rendering-perf.bench.js::Rendering perf (non-streaming)0%
8.6 ms8.6 ms
large-array [streaming]
benchmark/bench/rendering-perf.bench.js::Rendering perf (streaming)0%
149.9 ms149.7 ms
many-expressions (renderChild dispatch, escapeHTML)
benchmark/bench/rendering-perf.bench.js::Rendering perf (non-streaming)0%
23.8 ms23.8 ms
Rendering: streaming [true], .mdx file
benchmark/bench/render.bench.js::Bench rendering0%
573.5 ms572.9 ms
many-expressions [streaming]
benchmark/bench/rendering-perf.bench.js::Rendering perf (streaming)0%
23.5 ms23.6 ms
many-components [streaming]
benchmark/bench/rendering-perf.bench.js::Rendering perf (streaming)0%
8 ms8 ms
Rendering: streaming [false], .md file
benchmark/bench/render.bench.js::Bench rendering0%
1.1 ms1.1 ms
many-slots (eager slot prerendering)
benchmark/bench/rendering-perf.bench.js::Rendering perf (non-streaming)0%
5.4 ms5.4 ms
large-array (BufferedRenderer per child)
benchmark/bench/rendering-perf.bench.js::Rendering perf (non-streaming)0%
159.3 ms159.3 ms
static-heavy (markHTMLString baseline)
benchmark/bench/rendering-perf.bench.js::Rendering perf (non-streaming)0%
6 ms6 ms
many-head-elements (head dedup)
benchmark/bench/rendering-perf.bench.js::Rendering perf (non-streaming)0%
5 ms5 ms
Commits
Click on a commit to change the comparison rangeBase
main
ccecb8f
+10.42%
refactor(assets): improve cache validation code
53cc6b9
3 months ago
by oliverlynch-10.01%
chore: reword changeset to be more descriptive
27ff03d
3 months ago
by oliverlynch+0.08%
fix: add tests
d5ccbcc
3 months ago
by Princesseuh+10.85%
Merge branch 'main' into main
00a75dd
3 months ago
by Princesseuh-11.35%
Apply suggestions from code review
c2046e6
3 months ago
by Princesseuh