Postgres Pro
Downloads
Home   >   mailing lists

Re: [pgadmin4][Patch]: Fixed minor issue in auto discover server whenentry is blank in the config file - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgadmin4][Patch]: Fixed minor issue in auto discover server whenentry is blank in the config file
Date
Msg-id CA+OCxoxP=UJA_gd8oO4_2HVF6q8W9=YAKZ+BRNymKo4g9kv+pw@mail.gmail.com
Whole thread Raw
In response to [pgadmin4][Patch]: Fixed minor issue in auto discover server whenentry is blank in the config file  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
Tree view
Thanks, applied.

On Thu, Jun 21, 2018 at 8:32 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Hackers,

I have found one minor issue in auto discovery logic. getint function of ConfigParser throws exception if the value of the parameter is blank in the config file. Example: Port= 

Because of that exception pgAdmin4 unable to discover all the valid servers. Attached is the patch to fix that. Please review it. 

--
Akshay Joshi
Sr. Software Architect


Phone: +91 20-3058-9517
Mobile: +91 976-788-8246



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

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

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Minor fix to handle auto-discovery of servers withno
Next
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Allow dropping of Procedures in PG 11. Fixes #3423