Page MenuHomePhabricator

Correct two parameter strictness issues with file uploads
ClosedPublic

Authored by epriestley on Apr 10 2017, 10:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 15, 5:20 AM
Unknown Object (File)
Tue, Mar 12, 8:13 PM
Unknown Object (File)
Jan 31 2024, 2:24 AM
Unknown Object (File)
Jan 30 2024, 4:37 AM
Unknown Object (File)
Jan 5 2024, 12:12 AM
Unknown Object (File)
Dec 27 2023, 10:33 AM
Unknown Object (File)
Dec 26 2023, 7:51 PM
Unknown Object (File)
Dec 23 2023, 1:10 PM
Subscribers

Details

Summary

Fixes T12531. Strictness fallout from adding typechecking in D17616.

  • chunkedHash is not a real parameter, so the new typechecking was unhappy about it.
  • mime-type no longer allows null.
Test Plan
  • Ran arc upload --conduit-uri ... 12MB.zero on a 12MB file full of zeroes.
  • Before patch: badness, failure, fallback to one-shot uploads.
  • After patch: success and glory.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence added inline comments.
src/applications/files/conduit/FileUploadChunkConduitAPIMethod.php
70

Is this meant to be $params['mime-type']? Otherwise it looks like $mime_type is unused.

epriestley removed a reviewer: joshuaspence.
  • Actually set 'mime-type'.
This revision was automatically updated to reflect the committed changes.

I removed him accidentally by racing arc diff against his accept. T12109 is a rough description.

That is, he got removed here, when an arc diff with Reviewers: chad in the message took effect:

Screen Shot 2017-04-10 at 5.11.01 PM.png (80×450 px, 20 KB)