Re: [pgAdmin][RM5521] pgAdmin dump servers issue in desktop mode

2020-05-31 Thread Akshay Joshi
Thanks, patch applied. On Fri, May 29, 2020 at 5:25 PM Aditya Toshniwal < aditya.toshni...@enterprisedb.com> wrote: > Hi Akshay, > > Attached is the updated patch which hopefully works for all cases. :) > Whenever a wrong file, wrong path or some other non pgAdmin db file is > provided then it wi

Re: [pgAdmin][RM5521] pgAdmin dump servers issue in desktop mode

2020-05-29 Thread Aditya Toshniwal
Hi Akshay, Attached is the updated patch which hopefully works for all cases. :) Whenever a wrong file, wrong path or some other non pgAdmin db file is provided then it will throw a message - "SQLite pgAdmin config DB file is not valid". Please review. On Fri, May 29, 2020 at 11:28 AM Akshay Jo

Re: [pgAdmin][RM5521] pgAdmin dump servers issue in desktop mode

2020-05-28 Thread Akshay Joshi
Hi Aditya When I provide the wrong path of the SQLite DB file then it should not ask me an email address and password. python setup.py --sqlite-path "/Users/akshayjoshi/Downloads/pgadmin4-de.db" --dump-servers output.json Dumping servers with: User: pgadm...@pgadmin.org SQLite pgAdmin co

Re: [pgAdmin][RM5521] pgAdmin dump servers issue in desktop mode

2020-05-28 Thread Aditya Toshniwal
Hi Hackers, Attached is the updated patch. Please review. On Thu, May 28, 2020 at 5:38 PM Akshay Joshi wrote: > Hi Aditya > > When I run setup.py and specify the *--sqlite-path* parameter like below: > python setup.py --sqlite-path > "/Users/akshayjoshi/Downloads/pgadmin4-dev-env.db" --dump-ser

Re: [pgAdmin][RM5521] pgAdmin dump servers issue in desktop mode

2020-05-28 Thread Akshay Joshi
Hi Aditya When I run setup.py and specify the *--sqlite-path* parameter like below: python setup.py --sqlite-path "/Users/akshayjoshi/Downloads/pgadmin4-dev-env.db" --dump-servers output.json I got the following error: PermissionError: [Errno 13] Permission denied: '/var/lib/pgadmin' Please fix

[pgAdmin][RM5521] pgAdmin dump servers issue in desktop mode

2020-05-28 Thread Aditya Toshniwal
Hi Hackers, Attached is the patch to: 1) Fix an issue when dumping servers from a desktop pgAdmin app by providing an option --sqlite-path. This will allow the user to dump a specific config db file. 2) While there is an option for --sqlite-path now, the other paths for session, storage, etc. were