Re: [pgAdmin4][Patch] Feature #1447 SSH Tunnel

From: Anthony Emengo <aemengo(at)pivotal(dot)io>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch] Feature #1447 SSH Tunnel
Date: 2018-04-23 14:02:44
Message-ID: CAG8BBZMJNK22qA9jgjVA8K+b7s=ZV-=0DweC6wOvONZTHF=5PQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

For what it is worth, I manually verified that the feature worked, as well
as looked through the code.

I'd like to see end-to-end testing for regression sake, but it's hard to so
at this moment.

- Anthony and Joao.

On Mon, Apr 23, 2018 at 5:09 AM, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com
> wrote:

>
>
> On Mon, Apr 23, 2018 at 1:30 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Thu, Apr 19, 2018 at 6:56 PM, Anthony Emengo <aemengo(at)pivotal(dot)io>
>> wrote:
>>
>>> Hey Akshay
>>>
>>> This patch passed our test pipelines.
>>>
>>
>> Did you test the feature and//or review the code and tests? Passing the
>> tests is great, *if* the whole feature is covered (and the nature of this
>> patch will make that quite difficult, maybe impossible to do without
>> external infrastructure and config).
>>
>
> Agreed, it's been difficult to write test case to test the complete
> feature.
>
>>
>>
>>>
>>> Anthony and Victoria
>>>
>>> On Thu, Apr 19, 2018 at 1:48 AM, Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers
>>>>
>>>> I have implemented the SSH Tunnel support using
>>>> https://pypi.org/project/sshtunnel/ python package. Added "SSH Tunnel"
>>>> Tab in server dialog. This implementation supports user name /password and
>>>> private/public key combination with Passphrase to crate SSH Tunnel. I have
>>>> added regression test case to add server using SSH Tunnel options.
>>>>
>>>> The given python package(https://pypi.org/project/sshtunnel/) support
>>>> Python version *2.7, 3.4+*.
>>>> It uses Paramiko (Python implementation of SSHv2 protocol) which
>>>> actually drops support for Python 2.6. So I have added
>>>> *SUPPORT_SSH_TUNNEL* parameter in config.py which checks the python
>>>> version and set the flag accordingly. In case of Python 2.6, 3.0, 3.1, 3.2
>>>> and 3.3 control on the "SSH Tunnel" tab of server dialog will be disabled.
>>>>
>>>> Please review it, and if looks good please commit the code.
>>>>
>>>> --
>>>> *Akshay Joshi*
>>>>
>>>> *Sr. Software Architect *
>>>>
>>>>
>>>>
>>>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>>>
>>>
>>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
>
> --
> *Akshay Joshi*
>
> *Sr. Software Architect *
>
>
>
> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Anthony Emengo 2018-04-23 14:34:18 Re: [PATCH] [RM# 3290] Close button missing for the message window from the backend server
Previous Message Joao De Almeida Pereira 2018-04-23 13:49:12 [pgadmin4][patch] [GreenPlum] Display SQL of table with index #3306