Thread: Fix for issue RM1313 [pgadmin4]

Fix for issue RM1313 [pgadmin4]

From
Harshal Dhumal
Date:
Hi,


PFA patch for issue RM 1313.

Changes: Added connect now option in server create dialog.


-- 
Harshal Dhumal
Software Engineer

EnterpriseDB India: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment

Re: Fix for issue RM1313 [pgadmin4]

From
Dave Page
Date:
Thanks - committed with some minor changes:

- Move the password field under the user name.

- Remove the password requirement. The user might be using "trust" auth.

- Adjust the vertical positioning of the checkbox.

On Tue, Jun 14, 2016 at 2:18 PM, Harshal Dhumal
<harshal.dhumal@enterprisedb.com> wrote:
> Hi,
>
>
> PFA patch for issue RM 1313.
>
> Changes: Added connect now option in server create dialog.
>
>
> --
> Harshal Dhumal
> Software Engineer
>
> EnterpriseDB India: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Fix for issue RM1313 [pgadmin4]

From
Dave Page
Date:
On Tue, Jun 14, 2016 at 5:28 PM, Dave Page <dpage@pgadmin.org> wrote:
> Thanks - committed with some minor changes:
>
> - Move the password field under the user name.
>
> - Remove the password requirement. The user might be using "trust" auth.
>
> - Adjust the vertical positioning of the checkbox.

Hmm, this is actually broken:

1) I've left the password check in the Python code.

2) It seems like it saves the server info on the first connection
attempt. That means that if the connection fails, but the password is
not the issue (mis-typed address or username perhaps), correcting that
info doesn't allow the connection to proceed.

Because of 2, I haven't fixed 1 (you'll probably need to move it
around anyway). Essentially, we need to support password-less
connections, and if the connection fails, the user should be able to
immediately try again having changed any of the connection options.

Please take a look - and while you're in that code, please take a look
at #1363. It's probably a very simple fix.

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: Fix for issue RM1313 [pgadmin4]

From
Harshal Dhumal
Date:
Hi,


PFA attached patch for RM1313 updates and RM1363

-- 
Harshal Dhumal
Software Engineer

EnterpriseDB India: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On Tue, Jun 14, 2016 at 10:40 PM, Dave Page <dpage@pgadmin.org> wrote:
On Tue, Jun 14, 2016 at 5:28 PM, Dave Page <dpage@pgadmin.org> wrote:
> Thanks - committed with some minor changes:
>
> - Move the password field under the user name.
>
> - Remove the password requirement. The user might be using "trust" auth.
>
> - Adjust the vertical positioning of the checkbox.

Hmm, this is actually broken:

1) I've left the password check in the Python code.
Removed check
 

2) It seems like it saves the server info on the first connection
attempt. That means that if the connection fails, but the password is
not the issue (mis-typed address or username perhaps), correcting that
info doesn't allow the connection to proceed.
Fixed
 

Because of 2, I haven't fixed 1 (you'll probably need to move it
around anyway). Essentially, we need to support password-less
connections, and if the connection fails, the user should be able to
immediately try again having changed any of the connection options.

Added support for password less login
 

Please take a look - and while you're in that code, please take a look
at #1363. It's probably a very simple fix.

Fxied
 

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment

Re: Fix for issue RM1313 [pgadmin4]

From
Dave Page
Date:
Thanks, applied with some improvements to the error handling.

On Wed, Jun 15, 2016 at 11:15 AM, Harshal Dhumal
<harshal.dhumal@enterprisedb.com> wrote:
> Hi,
>
>
> PFA attached patch for RM1313 updates and RM1363
>
> --
> Harshal Dhumal
> Software Engineer
>
> EnterpriseDB India: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> On Tue, Jun 14, 2016 at 10:40 PM, Dave Page <dpage@pgadmin.org> wrote:
>>
>> On Tue, Jun 14, 2016 at 5:28 PM, Dave Page <dpage@pgadmin.org> wrote:
>> > Thanks - committed with some minor changes:
>> >
>> > - Move the password field under the user name.
>> >
>> > - Remove the password requirement. The user might be using "trust" auth.
>> >
>> > - Adjust the vertical positioning of the checkbox.
>>
>> Hmm, this is actually broken:
>>
>> 1) I've left the password check in the Python code.
>
> Removed check
>
>>
>>
>> 2) It seems like it saves the server info on the first connection
>> attempt. That means that if the connection fails, but the password is
>> not the issue (mis-typed address or username perhaps), correcting that
>> info doesn't allow the connection to proceed.
>
> Fixed
>
>>
>>
>> Because of 2, I haven't fixed 1 (you'll probably need to move it
>> around anyway). Essentially, we need to support password-less
>> connections, and if the connection fails, the user should be able to
>> immediately try again having changed any of the connection options.
>
>
> Added support for password less login
>
>>
>>
>> Please take a look - and while you're in that code, please take a look
>> at #1363. It's probably a very simple fix.
>
>
> Fxied
>
>>
>>
>> Thanks.
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>
>



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company