syndesisio / syndesis

A flexible, customizable, open source platform that provides core integration capabilities as a service.
https://syndesis.io/
Apache License 2.0
591 stars 205 forks source link

Public key authentication in SFTP connector #2660

Open zregvart opened 6 years ago

zregvart commented 6 years ago

This is a...


[x] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  
[ ] Documentation issue or request

The problem

For SFTP we support username and password authentication, we should also support public key authentication.

Expected behavior

Specify SSH private key instead of username and password to establish a connection to SFTP server.

Screenshot

Request and Response Data

API Endpoints and Schemas

Tasks involved / Steps to Reproduce

1. 2. 3. 4.

paoloantinori commented 6 years ago

First step is to verify the functionality exists at Camel level

oscerd commented 6 years ago

@zregvart do you mean by using the keyPair parameter in Camel-sftp component?

oscerd commented 6 years ago

Or using privateKey as bytes as camel-sftp provides?

zregvart commented 6 years ago

@oscerd I'm thinking of privateKey parameter for camel-sftp, we probably need a connector property that will accept the openssh format key and convert it to byte[] for the ftp2 component. I'd expect the users to provide the key in a text box by pasting something like:

-----BEGIN OPENSSH PRIVATE KEY-----
...
-----END OPENSSH PRIVATE KEY-----
oscerd commented 6 years ago

Understood, we'll need to set the preferredAuthentications in that case with public_key as first choice.

oscerd commented 6 years ago

@paoloantinori @heiko-braun I don't think this will go in this Sprint, I'll take care of this for the next one. Totally forget about it :-|

oscerd commented 5 years ago

Didn't have the time for this enhancement. Too much stuff also in sustaining team.

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had any activity since 60 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!

zregvart commented 5 years ago

Perhaps we should consider this in the next sprint. I think it makes sense for SFTP to support public key authentication.

oscerd commented 5 years ago

I completely forget about this again. Sorry.

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!