File Upload and Download changes for www site

indytechcook commented 9 years ago

Follow ups:

[x] Change all file uploads to private
[x] call handle call to /download when loading file
[ ] ~~use MPU AWS AP~~
[x] use S3 direct/pre signed URL MPU uploading

indytechcook commented 9 years ago

Upload link works now but S3 isn't configured correctly.

indytechcook commented 9 years ago

I had a bunch of issues with uploading the directly from the browser to S3. The issue is with the headers of the request to get the URL must match the headers to PUT the file.

I've kept the directive and service to upload the file on the www site but they aren't being used:

Follow up issue #446 logged.

To implement the new downloading of files:

call the file url + "download" on the challenge service to get the url
just download the file
upload the file via private

Remove the public header when uploading the file: https://github.com/appirio-tech/lc1-external-challenge-service/blob/master/controllers/challenges.js#L306

AngularJS Download Service on www: https://gitlab.com/topcoderinc/tc-site/blob/ed7bec0e6abb8d9abf43699da34285102be02f63/wp-content/themes/tcs-responsive/js/app/challenge-details/js/services/lc-download-service.js call getDownloadUrl
AngularJS Download Directive on www https://gitlab.com/topcoderinc/tc-site/blob/ed7bec0e6abb8d9abf43699da34285102be02f63/wp-content/themes/tcs-responsive/js/app/challenge-details/js/directives/lc-download-directive.js
AngualrJS Download Directive Implementation: https://gitlab.com/topcoderinc/tc-site/blob/ed7bec0e6abb8d9abf43699da34285102be02f63/wp-content/themes/tcs-responsive/ng-content-challenge-downloads.php#L6

cc @gaitonde

appirio-tech / lc1-project

File Upload and Download changes for www site #439