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

Error connecting to FTP with TLS #2596

Open
rfontes opened this issue May 11, 2017 · 3 comments
Open

Error connecting to FTP with TLS #2596

rfontes opened this issue May 11, 2017 · 3 comments
Assignees
Milestone

Comments

@rfontes
Copy link

rfontes commented May 11, 2017

I've been struggle with this error in my servers in TLS.

With FTP clients i connect successfully and in previous versions of komodo edit too.

The last releases (in v9) the problem start to accour.

Steps to Reproduce

Connect with FTP over SSL Explicit (server info shared in private)

Expected results

Connection

Actual results

Error: 'ssl.c:504: error:140770FC:SSL routines:SSL23GET_SERVER_HELLO:unknown protocol'

Platform Information

Komodo Edit, version 10.2.2, build 17703, platform win32-x86. Built on Sat Apr 29 02:34:29 2017.

Additional Information

[2017-05-11 16:01:35,961] [ERROR] koFTPSConnection: FTPS do_openSocket ERROR: _ssl.c:504: error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol
[2017-05-11 16:01:45,936] [WARNING] koScintillaSchemeService: applyScheme:: no indicator for name 'collab_remote_cursor_4'
[2017-05-11 16:01:45,936] [WARNING] koScintillaSchemeService: applyScheme:: no indicator for name 'collab_remote_cursor_5'
[2017-05-11 16:01:45,936] [WARNING] koScintillaSchemeService: applyScheme:: no indicator for name 'collab_remote_cursor_2'
[2017-05-11 16:01:45,937] [WARNING] koScintillaSchemeService: applyScheme:: no indicator for name 'collab_remote_cursor_3'
[2017-05-11 16:01:45,937] [WARNING] koScintillaSchemeService: applyScheme:: no indicator for name 'collab_remote_cursor_1'
[2017-05-11 16:01:45,937] [WARNING] koScintillaSchemeService: applyScheme:: no indicator for name 'collab_remote_change'
[2017-05-11 16:01:45,937] [WARNING] koScintillaSchemeService: applyScheme:: no indicator for name 'spelling_error'
[2017-05-11 16:01:45,937] [WARNING] koScintillaSchemeService: applyScheme:: no indicator for name 'collab_local_change'

@Naatan
Copy link
Member

Naatan commented May 11, 2017

Thanks @rfontes, I've removed the login information and shared it internally. Please keep in mind that our bug tracker is public. For the record this looks like it might be an issue with the server, not the client. Does the FTP connection work with other FTP clients?

@cgchoffman can you look into this one? Be sure to milestone it if it turns into a timesink.

@rfontes
Copy link
Author

rfontes commented May 11, 2017

yes i can login with filezilla anf cyberduck....

@th3coop
Copy link
Member

th3coop commented May 11, 2017

I've traced this to https://github.com/Komodo/Komodoedit/blob/master//src/components/koFTP.py#L649.
Which is Python ssl package. I've found reference to this issue in many locations on the internet.
The main solution appears to be to upgrade to Python 3.

I double checked and this is broken all the way back to Komodo 10.0.0. I didn't check 9 but I suspect that Komodo 9 is broken as well.

In case you're wondering, this is not related to our Paramiko upgrade. Paramiko only handles SFTP.

This would likely be a timesink as Upgrading to Python 3 for a Komodo 10 release isn't an option and I would have to try and figure out another option.

Possible alternative solution could be to install pyOpenSSL but I read other threads that said you should NOT need to do that.

I"m really surprised that no one else has hit this. FTPS connections are totally broken in Komodo since 10 was released? Or maybe this is related to an update to libraries as a result of the various OpenSSL bugs that have come out.

I'm tempted to milestone this for Komodo 11 since no one has reported it until now. I realize that sucks for you @rfontes. Don't take my comment as final say. Need to discuss internally over here.

@th3coop th3coop added this to the Perpetual milestone May 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants