-
Notifications
You must be signed in to change notification settings - Fork 363
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
Transient "451-Error during read from data connection" forces abort after max retries #322
Comments
I can confirm that I just ran into this as well. I ended up using sebastianpopp/ftp-action and that worked. I like SamKirkland/FTP-Deploy-Action because it has some nice features and useful output. Edit: I listed the wrong GH repo for my current solution. |
Is there a resolution for this issue? Having the same problem |
I made a PR #380, you can use yelper/FTP-Deploy-Action@master in the meantime. Here's my sample deploy file (I used |
See release |
awesome, I confirmed in live use. Thanks again!! |
Bug Description
It appears that the downstream dependency patrickjuchli/basic-ftp can abort a secure FTP connection too early when uploading "big" files around 45kB. The cause of this issue is the indirect dependency of this project on basic-ftp@4.6.6 instead of the fixed @^5 version.
A bug fix was introduced by #212 in patrickjuchli/basic-ftp. This bug reproduces consistently with my setup. I tried decreasing the timeout to no avail. Notice how the connection is closed before all data is transferred. My host is "knownhost" and I can provide a test FTPS account if requested.
Although your dependencies seem to pull in patrickjuchli/basic-ftp@^5.0.0 that contains this fix, the package-lock.json file in this repo still points to 4.6.6. The upstream projects' package-lock.json seems to be okay, so I think it's a deployment issue of your SamKirkland/FTP-Deploy package (the package registry SamKirkland/FTP-Deploy@1.2.2 notes ^4.6.2 for basic-ftp).
The dist.js code in this repro still uses the pre-fixed code of the original fix:
FTP-Deploy-Action/dist/index.js
Line 6341 in ec28745
Since this is a deployment issue, I think you need to re-issue a version for your upstream dependency and bump the version here too.
My Action Config
My Action Log
The text was updated successfully, but these errors were encountered: