serve assets gzipped #342

[gawicks]

No description provided.

[CLAassistant]

All committers have signed the CLA.

[codecov]

Codecov Report

Merging #381 (3aa6a6a) into main (e14038f) will increase coverage by 0.81%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##             main     #381      +/-   ##
==========================================
+ Coverage   50.26%   51.07%   +0.81%     
==========================================
  Files         117      117              
  Lines        5239     5258      +19     
==========================================
+ Hits         2633     2685      +52     
+ Misses       2355     2310      -45     
- Partials      251      263      +12     

Impacted Files	Coverage Δ
pkg/server/controller.go	45.74% <37.50%> (-0.12%)	⬇️
pkg/agent/upstream/remote/remote.go	62.32% <0.00%> (-2.89%)	⬇️
pkg/server/handler.go	2.55% <0.00%> (+1.91%)	⬆️
pkg/cli/flags.go	66.07% <0.00%> (+2.72%)	⬆️
pkg/cli/command.go	56.76% <0.00%> (+56.76%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0f491e...3aa6a6a. Read the comment docs.

[gawicks]

Setting this to WIP till I've written some tests.

[gawicks]

@petethepig I'd like some feedback on the use of klauspost/compress library.

[petethepig]

It looks good! I tried running it too and it seems to be working as expected. My original plan was to only enable it for everything in /assets, but then I thought about it and I figured this even better actually.

[petethepig]

@gawicks Oh, and regarding the library, I do think it makes sense to use it here.

[gawicks]

@petethepig Thanks! I've finished adding tests. The PR is ready for review.

[petethepig]

@gawicks Looks good! Cool use of DescribeTable, I didn't know you could do that :)

[gawicks]

Yay!