You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/10/15 19:10:54 UTC

[GitHub] [airflow] wolfier opened a new issue #11560: Allow for specification of cipher in SftpHook

wolfier opened a new issue #11560:
URL: https://github.com/apache/airflow/issues/11560


   <!--
   
   Welcome to Apache Airflow!  For a smooth issue process, try to answer the following questions.
   Don't worry if they're not all applicable; just try to include what you can :-)
   
   If you need to include code snippets or logs, please put them in fenced code
   blocks.  If they're super-long, please use the details tag like
   <details><summary>super-long log</summary> lots of stuff </details>
   
   Please delete these comment blocks before submitting the issue.
   
   -->
   
   **Description**
   
   I want to specify the cipher to use when creating a SftpHook. Currently this is not possible because there is not a way to propagate the value to CnOpts. 
   
   https://github.com/apache/airflow/blob/1.10.12/airflow/contrib/hooks/sftp_hook.py#L97
   https://github.com/apache/airflow/blob/1.10.12/setup.py#L378
   https://pysftp.readthedocs.io/en/release_0.2.9/cookbook.html
   
   **Use case / motivation**
   
   <!-- What do you want to happen?
   
   Rather than telling us how you might implement this solution, try to take a
   step back and describe what you are trying to achieve.
   
   -->
   
   I want to be able to access SFTP servers that have disabled access from connection with weak ciphers.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] mik-laj closed issue #11560: Allow for specification of cipher in SftpHook

Posted by GitBox <gi...@apache.org>.
mik-laj closed issue #11560:
URL: https://github.com/apache/airflow/issues/11560


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] wolfier commented on issue #11560: Allow for specification of cipher in SftpHook

Posted by GitBox <gi...@apache.org>.
wolfier commented on issue #11560:
URL: https://github.com/apache/airflow/issues/11560#issuecomment-709562764


   Here's some [unit tests](https://github.com/ryhsiao/pysftp/blob/master/tests/test_ciphers.py) in pysftp using copts and ciphers.
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] mik-laj commented on issue #11560: Allow for specification of cipher in SftpHook

Posted by GitBox <gi...@apache.org>.
mik-laj commented on issue #11560:
URL: https://github.com/apache/airflow/issues/11560#issuecomment-712141600


   @wolfier Looks interesting. Can I assign you to this issue? We are open to pull requests from external contributors.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org