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.

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

Slow uploads on SFTP #1455

Closed
matisiekpl opened this issue Feb 23, 2020 · 7 comments
Closed

Slow uploads on SFTP #1455

matisiekpl opened this issue Feb 23, 2020 · 7 comments

Comments

@matisiekpl
Copy link

Hello.
I'm experiencing low upload speed, while download speed is very OK.
I've installed mcrypt.
What should I do to increase the speed of uploads?
Thanks, Matt

@terrafrost
Copy link
Member

What version of phpseclib are you using?

If you're using the new 2.0.24 that I released last night then there's this new change in that version:

333e2e4#commitcomment-36576462

I'm kinda wondering if that could be causing you issues. If it is then the following will revert you back to the last version (2.0.23):

composer require phpseclib/phpseclib:"~2.0, !=2.0.24"

lmk if that works for you and if so then I'll consider doing a new release that doesn't have that change in it.

Thanks!

@BBrunekreeft
Copy link

I experienced the exact same problem. Downgrading to 2.0.23 resolved the speed problems.

@BBrunekreeft
Copy link

I can confirm that there are no speed issues with version 2.0.25.
Thanks!

@terrafrost
Copy link
Member

Thanks!

I want to make another commit in an attempt to speed uploads up tonight. Maybe y'all can test it out?

I tested it with a 1GB file and it was able to upload it to localhost in 17s whereas without any changes it took 30s.

With the #1425 -inspired changes I probably did a 100MB file when I was testing them. It seemed to hang when I uploaded a 1GB file with those changes last night.

Thanks!

@BBrunekreeft
Copy link

I'll be happy to test it out.

@terrafrost
Copy link
Member

I've gone ahead and merged this. If there are further speed issues create another ticket.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants