DeprecationWarning: Ftp.raw[user](args): Use Ftp.raw('user args') instead.

Question

DeprecationWarning: Ftp.raw[user](args): Use Ftp.raw('user args') instead.

Closed this issue 7 years ago · 17 comments

When I use ftp.auth method, this deprecation warning appear. is there anyway to do authentication. my code is following.

const connect = (server) => new Promise((resolve,reject)=>{
	
	// create new ftp server with r	
	const ftp = new JSFtp({
		host: server.host,
		port: server.port || 21,
		user: server.user,
		pass: server.pass,
		debugMode: true
	});	
		

	ftp.on('error',function(err){
		if(err.code == 530 ) reject({status: true, err: 'invalid user credential'});
		if(err.code == 'ENOTFOUND') reject({status: true, err: 'invalid host'});
		reject({status: true,err:'criteria is not correct. cannot connect to server' });
	});

	ftp.on('connect',function(err){
		
		ftp
			.auth(server.user,server.pass,(err,data) => {
				if(err)	reject({ status: true, err: 'Error: 530 Login incorrect'});

				resolve(ftp);
			})
		// resolve(ftp);
	})
});

adamgedney commented 7 years ago

Same here

👍4

Answer 1 · 2017-02-05T18:31:07.000Z

It happens the same with me :(

Answer 2 · 2017-02-08T13:40:32.000Z

This is caused by the auth function itself for user look at line 334, for password its line 340 and for type it's line 349.
They should be changed to self.raw('user ' + user, function(..., self.raw('pass' + pass, function(... and self.raw('type I', function(....

Answer 3 · 2017-02-24T20:57:47.000Z

This was already fixed and it is merged on master: 35a8edd. Could anyone release a new version so we stop receiving these warnings, please?

Answer 4 · 2017-03-11T07:15:16.000Z

Bump. Can we have a new release, please?

Answer 5 · 2017-03-12T18:30:18.000Z

Receiving this warning too. Bump.

Answer 6 · 2017-03-22T23:00:51.000Z

Receiving. Bump.

Answer 7 · 2017-05-02T19:05:52.000Z

Although this has been closed the lines referred to by @svenbw remain unaltered in the version distributed via npm. I think the new code needs to be npm published