Page MenuHomeSolus

nginx: Update to 1.21.0
ClosedPublic

Authored by ReillyBrogan on Wed, Jun 2, 6:03 PM.

Details

Summary

Security Updates:

Other release notes available here

Test Plan
  • Tested a simple static site
  • Tested as a reverse proxy to a locally hosted app

Diff Detail

Repository
R2166 nginx
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ReillyBrogan created this revision.Wed, Jun 2, 6:03 PM
ReillyBrogan requested review of this revision.Wed, Jun 2, 6:03 PM
JoshStrobl requested changes to this revision.Wed, Jun 2, 11:19 PM
JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
files/nginx.conf
32 ↗(On Diff #27043)

This isn't really meaningful to have enabled by default in our nginx. People shouldn't be serving web pages with nginx under Solus unless it is for local development (where providing compressed assets is less of a concern).

This revision now requires changes to proceed.Wed, Jun 2, 11:19 PM
  • Disable gzip for default web page
JoshStrobl accepted this revision.Thu, Jun 3, 7:27 AM

LGTM, thanks!

This revision is now accepted and ready to land.Thu, Jun 3, 7:27 AM
This revision was automatically updated to reflect the committed changes.