add an option to disable concurrent reads

[drakkan]

Fixs #345

[corey-fuller]

@puellanivis Is there a plan for this to be merged soon? My team has some issues with downloading files from a new SFTP server that this change may resolve.

[drakkan]

@puellanivis Is there a plan for this to be merged soon? My team has some issues with downloading files from a new SFTP server that this change may resolve.

Hi, you could test this patch and tells us if it works, we don't have a read once SFTP server to test it ourself, thank you

[corey-fuller]

It actually does work for our case. Thanks for checking with us. We haven’t tested every case, and don’t know a lot about how this works behind the scenes, but for our use case, it is working. We had an issue that this fix resolves with connecting to an IBM Sterling File Gateway SFTP server. Thanks, Corey

…

On Thu, Mar 4, 2021 at 15:18 Nicola Murino ***@***.***> wrote: @puellanivis <https://github.com/puellanivis> Is there a plan for this to be merged soon? My team has some issues with downloading files from a new SFTP server that this change may resolve. Hi, you could test this patch and tells us if it works, we don't have a read once SFTP server to test it ourself, thank you — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#408 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ATC3KHDYOOK4TMF34EIRTHLTB72KRANCNFSM4YC647RA> .

[drakkan]

It actually does work for our case. Thanks for checking with us. We haven’t tested every case, and don’t know a lot about how this works behind the scenes, but for our use case, it is working. We had an issue that this fix resolves with connecting to an IBM Sterling File Gateway SFTP server. Thanks, Corey
…
On Thu, Mar 4, 2021 at 15:18 Nicola Murino @.***> wrote: @puellanivis https://github.com/puellanivis Is there a plan for this to be merged soon? My team has some issues with downloading files from a new SFTP server that this change may resolve. Hi, you could test this patch and tells us if it works, we don't have a read once SFTP server to test it ourself, thank you — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#408 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATC3KHDYOOK4TMF34EIRTHLTB72KRANCNFSM4YC647RA .

Thank you for confirming, I'll do the change requested by @puellanivis and then I think this patch could be merged

[corey-fuller]

Thank you both so much for getting this out there so quickly @puellanivis & @drakkan.
This is super helpful for our team. :)