You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by GitBox <gi...@apache.org> on 2020/09/17 13:18:37 UTC
[GitHub] [cordova-plugin-file-transfer] ssledorze edited a comment on issue #281: Downloading a lot of big files in parallel leads to stack overflows (Range Error)
ssledorze edited a comment on issue #281:
URL: https://github.com/apache/cordova-plugin-file-transfer/issues/281#issuecomment-694226380
@breautek I agree about the nesting calls.
But I think changing the communication scheme between the Plugins and Cordova, while the best move, is a very wide change to Cordova and I think won't happen over night (I may be wrong)..
The way the PR is done is not that intrusive but I agree it's an additional feature to maintain.
Let me know your thinking; I need the fix to hit prod ASAP (bloodbath there..)
I think I can live with a fork to maintain but ideally we would highlight the issue to other devs and offer a path to a non crashing solution.
Actually without a solution in that Plugin they would simply leave it for another one or fork as there's no other workaround ATM (even if the wrong design is on cordova's side).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org