From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Jacek Wielemborek <d33tah(at)gmail(dot)com>, "pgadmin-hackers(at)postgresql(dot)org" <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: SSH error messages not shown |
Date: | 2015-07-07 05:05:49 |
Message-ID: | CANxoLDd9=d9Tm1nZX+8rR4Gd+Nn7WvJEQwrfvQE5-T1u9zy9rA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Jacek
Have you test the patch for SSH Tunnel error message?
On Mon, Jun 15, 2015 at 12:17 PM, Akshay Joshi <
akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
> Hi Jacek
>
> On Mon, Jun 15, 2015 at 10:40 AM, Akshay Joshi <akshay(dot)joshi(at)enterprisedb
> .com> wrote:
>
>> Sure.
>>
>> On Sun, Jun 14, 2015 at 3:13 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Akshay, can you look into this please?
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK:http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>> On 13 Jun 2015, at 16:22, Jacek Wielemborek <d33tah(at)gmail(dot)com> wrote:
>>>
>>> Hi,
>>>
>>> When an libssh2 call fails, pgAdmin III just shows the error code and
>>> nothing else. Consider the following patch - this is what I created
>>> while wondering why my hostkeys don't work:
>>>
>>> diff --git a/pgadmin/utils/sshTunnel.cpp b/pgadmin/utils/sshTunnel.cpp
>>> index fd80214..e15f764 100644
>>> --- a/pgadmin/utils/sshTunnel.cpp
>>> +++ b/pgadmin/utils/sshTunnel.cpp
>>> @@ -201,8 +201,13 @@ bool CSSHTunnelThread::Initialize()
>>> #endif
>>> if (rc)
>>> {
>>> -
>>> LogSSHTunnelErrors(wxString::Format(_("SSH error: Authentication by
>>> identity file failed with error code %d"), rc), GetId());
>>> + char* errmsg;
>>> + int errmsg_len;
>>> + libssh2_session_last_error(m_session,
>>> &errmsg, &errmsg_len, 0);
>>> + wxString errmsg_s(errmsg, wxConvLibc);
>>> +
>>> LogSSHTunnelErrors(wxString::Format(_("SSH error: Authentication by
>>> identity file failed with error code %d [%s]"), rc, errmsg_s.c_str()),
>>> GetId());
>>> Cleanup();
>>> + free(errmsg);
>>> return false;
>>> }
>>> }
>>>
>>> Perhaps we could use this kind of routines wherever rc is being checked?
>>>
>>> I have reviewed your patch and it looks good to have error
> message with error number. I have made some changes in the "
> LogSSHTunnelErrors" function to avoid duplicate logic for each error
> message. I have added one default parameter of the session object in the "
> LogSSHTunnelErrors" function. If we would like to log error message with
> error number, will have to pass session object. Attached is the modified
> patch, can you please check it is working or not for your test case.
>
>>
>>>
>>> Cheers,
>>> d33tah
>>>
>>>
>>
>>
>> --
>> *Akshay Joshi*
>> *Principal Software Engineer *
>>
>>
>>
>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>
>
>
>
> --
> *Akshay Joshi*
> *Principal Software Engineer *
>
>
>
> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>
--
*Akshay Joshi*
*Principal Software Engineer *
*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Jacek Wielemborek | 2015-07-07 10:00:04 | Re: SSH error messages not shown |
Previous Message | Sanket Mehta | 2015-07-06 12:41:42 | Re: PATCH: fix size check in dlgProperty |