Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

File uploads #15

Merged
merged 12 commits into from
Jul 15, 2015
Merged

File uploads #15

merged 12 commits into from
Jul 15, 2015

Conversation

roydejong
Copy link
Owner

Ticket #8

@roydejong
[#8#] Draft first version of FileUpload class
c12b687
@roydejong
[#8#] FileUpload: saveTo() should always return false if an error is …
a9dc400 
…present or the source file is invalid or empty (general security hardening)
@roydejong
[#8#] FileUpload: createFromFileArray() implementation
0234cea
@roydejong
[#8#] FileUpload: Implement getFileSize()
ca56ff0
@roydejong
[#8#] FileUpload: Add support for calling saveTo() multiple times (mo…
1fb647f 
…ve the first time using the safe function, and make additional copies the second time) by tracking the "current" location of files.
@roydejong
[#8#] FileUpload: Determine file size based on the temporary file on …
aeaf776 
…init to aid performance and ensure accuracy after moving the file
@roydejong
[#8#] FileUpload: Implement into Request class
0dcd3b8
@roydejong
[#8#] FileUpload: Add unit tests for file upload processing and valid…
ae858d5 
…ating
@roydejong
[#8#] FileUpload tests: Properly mock file uploading handling, increa…
a13996b 
…se coverage to 100%
@roydejong
[#8#] FileUpload: Harden against arrays being submitted and causing w…
d2539d3 
…arnings
@roydejong
[#8#] Support multiple file uploads
6f8d216
@roydejong
Travis: Fix strict standards warnings being raised in RequestTest
85c5cfa
roydejong added a commit that referenced this pull request Jul 15, 2015
@roydejong
Merge pull request #15 from roydejong/8-file-uploads
6e4ec35 
File uploads
@roydejong roydejong merged commit 6e4ec35 into master Jul 15, 2015
@roydejong roydejong deleted the 8-file-uploads branch July 16, 2015 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@roydejong