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

Depreciation Warning #117

Open
opencityint opened this issue Dec 11, 2016 · 7 comments
Open

Depreciation Warning #117

opencityint opened this issue Dec 11, 2016 · 7 comments

Comments

@opencityint
Copy link

Uploading from command line
grunt ftp
success! but warnings which I don't remember from a while back(?) .ftppass and inline credentials all standard with double quotes.

(node:78761) DeprecationWarning: Ftp.rawuser: Use Ftp.raw('user args') instead.
(node:78761) DeprecationWarning: Ftp.rawpass: Use Ftp.raw('pass args') instead.
(node:78761) DeprecationWarning: Ftp.rawtype: Use Ftp.raw('type args') instead.
(node:78761) DeprecationWarning: Ftp.rawcwd: Use Ftp.raw('cwd args') instead.
(node:78761) DeprecationWarning: Ftp.rawquit: Use Ftp.raw('quit args') instead.

thanks in advance

@yepes
Copy link

yepes commented Jan 16, 2017

Getting the same messages here

@maelfrize
Copy link

Same here

Thanks for the great library 👍

@Kelderic
Copy link

Kelderic commented Aug 3, 2017

This is still happening. It has been fixed upstream. We just need to edit line 198 of the main tasks file:

ftp.raw.quit(function (err) {

Change to:

ftp.raw('quit', function (err) {

@spoji
Copy link
Contributor

spoji commented Nov 7, 2017

Adding to the comment of @Kelderic , I've created a PR that fix all the warnings for me. You can take a look at it and change the files locally while we are waiting for @zonak to comment on the PR.

Cheers!

@Kelderic
Copy link

Kelderic commented Nov 7, 2017

Unfortunately, I'm pretty sure that this project has been abandoned, @spoji.

@spoji
Copy link
Contributor

spoji commented Nov 7, 2017

Yeah... Last updates were 3 years ago. The pull request should at least help others having the same issue.

@spoji
Copy link
Contributor

spoji commented Nov 7, 2017

I stand corrected. The PR has been merged back and npm has been updated to 0.2.0 which include the fix.

https://www.npmjs.com/package/grunt-ftp-deploy

@zonak This issue can be closed.

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

5 participants