Details

Reviewers

Duranzed
dereckson

Commits

rOPS5f294f6b900b: Increase upload limit to 100M for wikis (1/2 - nginx)

Summary

MediaWiki allows by default uploads up to 100 MB. This makes sense nowadays when
trying to upload larger files, like high-resolution pictures from a smartphone.

Default value for nginx is much lower.

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Skipped

Unit

No Test Coverage

Branch

mediawiki-only (branched from main)

Build Status

Buildable 6315
Build 6599: arc lint + arc unit

Event Timeline

Duranzed requested review of this revision.Thu, Feb 12, 11:30

Duranzed created this revision.

Harbormaster completed remote builds in B6311: Diff 10238.Thu, Feb 12, 11:30

Duranzed accepted this revision.Thu, Feb 12, 11:42

This revision is now accepted and ready to land.Thu, Feb 12, 11:42

Duranzed requested review of this revision.Thu, Feb 12, 11:43

Duranzed added a reviewer: dereckson.

Duranzed edited the summary of this revision. (Show Details)Thu, Feb 12, 11:52

Update of the file to match nginx configuration

Harbormaster completed remote builds in B6312: Diff 10239.Thu, Feb 12, 12:55

Updated file to match nginx

Harbormaster completed remote builds in B6313: Diff 10240.Thu, Feb 12, 12:58

dereckson mentioned this in D3951: Increase upload limit to 100M for wikis (2/2 - PHP).Thu, Feb 12, 20:55

dereckson retitled this revision from Add Nginx directive to accept files less 100M to Increase upload limit to 100M for wikis (1/2 - nginx).

dereckson edited the summary of this revision. (Show Details)

dereckson edited the test plan for this revision. (Show Details)

dereckson requested changes to this revision.Thu, Feb 12, 21:00

dereckson added inline comments.

roles/saas-mediawiki/nginx/files/includes/mediawiki-root
24–26	It would make more sense to first put general rules valid for each blocks, then the individual location blocks: ` root ... client_max... location ...
roles/saas-mediawiki/nginx/files/includes/mediawiki-wiki
23	Extraneous line Tip: you can use `git add -p` when you edit several files to only take content values, not whitespaces.