Page MenuHomeSolus

nginx: Update to 1.21.0

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



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

R2166 nginx
Automatic diff as part of commit; lint not applicable.
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.
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.