SF-3513 Add pre-compressed Brotli support to Kestrel#3722
SF-3513 Add pre-compressed Brotli support to Kestrel#3722RaymondLuong3 merged 1 commit intomasterfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3722 +/- ##
==========================================
- Coverage 81.45% 81.32% -0.13%
==========================================
Files 620 620
Lines 38960 39023 +63
Branches 6336 6341 +5
==========================================
+ Hits 31734 31735 +1
- Misses 6256 6318 +62
Partials 970 970 ☔ View full report in Codecov by Sentry. |
99a1930 to
12c7e14
Compare
pmachapman
left a comment
There was a problem hiding this comment.
@pmachapman resolved 1 discussion and dismissed @github-advanced-security[bot] from a discussion.
Reviewable status: 0 of 3 files reviewed, all discussions resolved.
6682786 to
0590fe2
Compare
RaymondLuong3
left a comment
There was a problem hiding this comment.
This is working from my testing. The static files are compressed to the br format.
@RaymondLuong3 reviewed 3 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on pmachapman).
This PR adds Brotli support to Kestrel, as Apache passes static file requests straight through to Kestrel.
The easiest place to test is QA, but if you want to test locally, run the Docker image:
dotnet build cd src/Docker docker compose build docker compose upAccessing Scripture Forge, and viewing network requests for static files will confirm that Brotli compressed versions are returned.
This change is