worksofliam / ILEditor

IBM i development environment (IDE)
https://worksofbarry.com/ileditor/
GNU General Public License v3.0
86 stars 29 forks source link

Connection over SSH/SFTP instead of FTP #99

Open worksofliam opened 5 years ago

worksofliam commented 5 years ago

Hi

It has been brought to my attention that the implementation of SSH & SFTP into ILEditor as the connection type is actually fairly straight forward. I have been able to change some parts of the editor to use SSH & SFTP and it works perfectly (and faster in some cases).

There are benefits to using SSH/SFTP too.


This is not a small change though. Here is a list of things that I think will need changing:

In fact, there could be a lot of changes that a complete refactor of the whole project has been on my mind.


Sadly, this is a lot of work and not something I can dedicate a lot of my time too right now. I can dedicate more time if people think this is important.

Please also leave any further constructive thoughts or ideas you have below.

Thanks, Liam Barry πŸ˜ƒ

ibmi-effinsoftware commented 5 years ago

It's absolutely the right way to go and should be the highest priority

danielgoodwin commented 5 years ago

IMO, You are looking at a couple of different issues.

The use of secure communication is probably a must nowadays and use of SFTP/SSH is one avenue for that.

The CCSID handling and SRCDAT are different issues but need additional base support for them to happen and you could address these subsequent to secure communications support.

Might I add the possibility of DDM into the mix as per the blog. It can provide SSL connections , CCSID conversion handling and access to SRCDAT/SRCSEQ fields as well. I've done a POC for DDM connections using the jtopenlite jar file for use in .Net via IKVM. Admittedly it does not yet address the other concerns but the functionality is there to progress.

worksofliam commented 5 years ago

@danielgoodwin

Thanks for your response. Agreed about the use of a secure communication. I've just been taking a look at your fork and I think the use of the IBMiTransport interface will solve a few issues - we can then perhaps let the user choose the connection type they want to use (including your DDM implementation!). How exciting!

All I am saying is that it seems that using SFTP over FTP might be a good option for the future - in fact, I'd be quite happy with deprecating the FTP method in ILEditor once DDM or SSH is more stable.

All the best, Liam

ataylorkato commented 5 years ago

I think secure FTP is absolutely the way to go by default. However, I think deprecating FTP entirely would be a misstep. As much as it's not ideal, there's still a lot of boxes out there that I run into that FTP would be the only method available to use ILEditor.

sanotto commented 5 years ago

I can help with IBMi.cs and IBMiUtils.cs, if you like.I've explored the issue and found sftp and ftps lacking the foundational feature for ile Editor ..."quote rcmd" so it need to be replaced by ssh and scp the problem being the lack of "auto translation" of text files. What about ILE Editor 2 ? I've read somewhere that was going to be web based, is that true?. What are your plans with Ile Editor 2?

worksofliam commented 5 years ago

@sanotto Thanks for your reply!

ILEditor 2 is a closed source project (https://ileditor.online) which is a re-write of ILEditor but in Electron and also cross-platform. It is not yet released.

volmcat commented 4 years ago

@worksofliam have you given this any more thought? or have any of the forks been able to get SSH/SFTP functional working? Would love to see this feature implemented.