froemken / dropbox

Provides a DropBox driver for the TYPO3 File Abstraction Layer
Other
25 stars 9 forks source link

Fix DumpFile needing excessive memory #6

Closed dreistromlandMf closed 5 years ago

dreistromlandMf commented 6 years ago

Downloading big files is not working because the file is saved as a whole into memory.

This commit will write the file directly into php://output without being saved into memory.

Dropbox-PHP-SDK is updated to a fork based on 0.2.1. The PR to upstream has been made: https://github.com/kunalvarma05/dropbox-php-sdk/pull/122

froemken commented 5 years ago

Thank you @tritumMf

Sorry, I have seen your pull request to late. In the meantime I have made fal_dropbox compatible to TYPO3 8 and 9. So, too many files have changed. I have copied your 5 or 6 rows into the new version, but I can not merge your pull request.

Version 3.0.1

Hope that is OK for you

Stefan

dreistromlandMf commented 5 years ago

Sure! But if I remember correctly, the dropbox_php_sdk did make some problems without the mentioned PR being merged, but that project appears to be inactive. I'm not quite sure what to do here and it's been almost a year, but the 5 or 6 rows probably won't do the trick.