[pgadmin-hackers] QT dependances of the web version?

2017-04-21 Thread Josh Berkus
Dave, all: Does the web version of pgadmin4 rely on QT somehow? The reason I ask is that, when you install pgadmin4 on a server, installing the required QT modules drags in all of Xorg/Wayland, which is around 250MB of packages which aren't needed or wanted on a server. If not, I'll look for a

Re: [pgadmin-hackers] Install of pgadmin4 from package fails ...

2017-04-21 Thread Josh Berkus
On 04/21/2017 02:14 PM, Josh Berkus wrote: > On 04/21/2017 01:57 AM, Devrim Gündüz wrote: >> On RHEL 7 / Fedora 25: >> >> * systemctl enable pgadmin4-v1.service; systemctl start pgadmin4-v1.service >> * cp /etc/httpd/conf.d/pgadmin4-v1.conf.sample /etc/httpd/conf.d/pgadmin4- >> v1.conf >> *

[pgadmin-hackers][patch] Dependents and Dependencies in GreenPlum

2017-04-21 Thread Joao Pedro De Almeida Pereira
Hi Hackers, We found out that when you are connected to a GreenPlum database and try to get Dependents and Dependencies of an object the application was returning a SQL error. This patch splits the SQL query used to retrieve the Dependents, Dependencies, and Roles SQL file into multiple

Re: [pgadmin-hackers][patch] Update chrome driver to support chrome version 58 in tests

2017-04-21 Thread Joao Pedro De Almeida Pereira
Hi Hackers, We recreated the patch and should work now. The problem looks like an issue with pip. I just opened an issue in their github . Thanks Joao & Oliver On Fri, Apr 21, 2017 at 5:20 AM, Dave Page wrote: > Hi, > >

Re: [pgadmin-hackers] Pains and thoughts about refactoring the Tree Menu using React

2017-04-21 Thread Joao Pedro De Almeida Pereira
Hi Hackers, The attachment was missing in the previous email, so there you go. Note: We had to remove the extension of the file due to security reasons, just add a .js Thanks Joao & Oliver On Fri, Apr 21, 2017 at 2:47 PM, Joao Pedro De Almeida Pereira < jdealmeidapere...@pivotal.io> wrote: >

[pgadmin-hackers] Pains and thoughts about refactoring the Tree Menu using React

2017-04-21 Thread Joao Pedro De Almeida Pereira
Hi Hackers, After a conversation with Dave we believe that we need to provide more context on our pains and what we propose as a first step for implementing the Tree Menu using React. The pain: Here is a quick description of what we think we understand and where we got stuck while doing our

Re: [pgadmin-hackers] Some questions about configuration and the pgadmin4-v1-web package

2017-04-21 Thread Josh Berkus
Devrim, > * cp /etc/httpd/conf.d/pgadmin4-v1.conf.sample /etc/httpd/conf.d/pgadmin4- > v1.conf One more question: the sample .conf doesn't have the threads-and-1-process configuration recommended in the pgadmin docs. Is there a reason it doesn't need it? -- Josh Berkus Containers & Databases

Re: [pgadmin-hackers] Some questions about configuration and the pgadmin4-v1-web package

2017-04-21 Thread Josh Berkus
On 04/21/2017 01:57 AM, Devrim Gündüz wrote: > > Hi Josh, > > On Thu, 2017-04-20 at 13:50 -0700, Josh Berkus wrote: >> If I install the pgadmin4-v1-web package from Yum, it's a bit unclear on >> what actual configuration needs to be done after that. Things I know >> need to be done: >> >> *

Re: [pgadmin-hackers][patch] Move to Alembic migration system

2017-04-21 Thread Joao Pedro De Almeida Pereira
Hello Hackers, We review the patch, just noticed a spelling issue so we regenerated the patch. Thanks Joao & Oliver On Fri, Apr 21, 2017 at 1:21 AM, Ashesh Vashi wrote: > Hi Joao & Oliver, > > On Fri, Apr 21, 2017 at 3:39 AM, Joao Pedro De Almeida Pereira < >

[pgadmin-hackers] [pgAdmin4][Patch]: RM #2214 PostgreSQL 10.0 compatibility

2017-04-21 Thread Akshay Joshi
Hi All Please find the patch to support PostgreSQL 10 compatibility, though it was not completed as SCRAM authentication support is still remaining to change the password. Patch contains following: - Changed the function name as it was change in v10. - Changed the SQL to get the sequence

[pgadmin-hackers] [pgAdmin4][PATCH] To fix the issue of EXPLAIN option with IE

2017-04-21 Thread Murtuza Zabuawala
Hi, Please find the patch to fix the issue while using ANALYZE option in IE. *Issue:* We were using .startsWith() function to check the sql if it is already starts with explain clause or not, which was added in commit: c758920890309e50f47a9427039cf735ddf04d37 But IE do not support .startsWith()

Re: [pgadmin-hackers][patch] Update chrome driver to support chrome version 58 in tests

2017-04-21 Thread Dave Page
Hi, Unfortunately I've had to revert this as the Jenkins CI builds started failing on all branches. It appears that the --install-option for chromerdriver_installer is also passed to pyperclip, which then barfs on it:

[pgadmin-hackers] pgAdmin 4 commit: Revert "Update Chromedriver version for Chrome 58."

2017-04-21 Thread Dave Page
Revert "Update Chromedriver version for Chrome 58." This reverts commit 70832286bcee0a02eb6acca67a58878954fda861. The Jenkins CI doesn't like this at all. Branch -- master Details ---

[pgadmin-hackers] pgAdmin 4 commit: Revert "Ensure PIP is the latest version when running

2017-04-21 Thread Dave Page
Revert "Ensure PIP is the latest version when running the CI tests." This reverts commit bf1f4aefa686554fa24ac2fbcf1457aa9de4eccf. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=f6ffaaff98f117e69c34c7bc30dd7f61b971c9ae Modified Files

[pgadmin-hackers] pgAdmin 4 commit: Ensure PIP is the latest version when running the CI

2017-04-21 Thread Dave Page
Ensure PIP is the latest version when running the CI tests. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=bf1f4aefa686554fa24ac2fbcf1457aa9de4eccf Modified Files -- ci/run_python_tests.sh | 1 + 1 file changed, 1 insertion(+)

[pgadmin-hackers] pgAdmin 4 commit: Enable horizontal scrolling in the treeview. Fixes #2

2017-04-21 Thread Dave Page
Enable horizontal scrolling in the treeview. Fixes #2339 Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=618d9bc7a96a91686c562c04b6885a437db05c4f Author: Khushboo Vashi Modified Files --

Re: [pgadmin-hackers] [pgAdmin4][Patch]: RM #2339 - Horizontal scrolling in the treeview

2017-04-21 Thread Dave Page
Thanks, patch applied! On Fri, Apr 21, 2017 at 5:10 AM, Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi, > > Please find the attached updated patch. > I have tested this patch on Ubuntu (Chrome & Firefox) and Mac (Chrome). > > Thanks, > Khushboo > > On Tue, Apr 18, 2017 at 7:25 PM,

Re: [pgadmin-hackers] Some questions about configuration and the pgadmin4-v1-web package

2017-04-21 Thread Devrim Gündüz
Hi Josh, On Thu, 2017-04-20 at 13:50 -0700, Josh Berkus wrote: > If I install the pgadmin4-v1-web package from Yum, it's a bit unclear on > what actual configuration needs to be done after that. Things I know > need to be done: > > * running setup.py > * copying the

[pgadmin-hackers] pgAdmin 4 commit: Update Chromedriver version for Chrome 58.

2017-04-21 Thread Dave Page
Update Chromedriver version for Chrome 58. Note: If you had previously installed the old driver you need to uninstall and install again it. $ pip uninstall chromedriver_installer $ pip install -r regression/requirements.txt Branch -- master Details ---

Re: [pgadmin-hackers][patch] Update chrome driver to support chrome version 58 in tests

2017-04-21 Thread Dave Page
Thanks, patch applied. On Thu, Apr 20, 2017 at 11:03 PM, Joao Pedro De Almeida Pereira < jdealmeidapere...@pivotal.io> wrote: > Hi Hackers, > > With the update to version 58 of chrome the version of the chrome driver > of Selenium need to be updated to version 2.29. > > This patch does that

Re: [pgadmin-hackers] Some questions about configuration and the pgadmin4-v1-web package

2017-04-21 Thread Dave Page
On Thu, Apr 20, 2017 at 9:50 PM, Josh Berkus wrote: > pgAdmin team, Devrim: > > If I install the pgadmin4-v1-web package from Yum, it's a bit unclear on > what actual configuration needs to be done after that. Things I know > need to be done: > > * running setup.py > * copying

Re: [pgadmin-hackers] [GENERAL] pgAdmin4 needs information of v10 SCRAM authentication

2017-04-21 Thread Akshay Joshi
On Fri, Apr 21, 2017 at 12:20 PM, Michael Paquier wrote: > On Fri, Apr 21, 2017 at 3:43 PM, Akshay Joshi > wrote: > >Thanks Michael, will check this. > > One thing I forgot to mention... Both StoredKey and ServerKey are now >

Re: [pgadmin-hackers] [GENERAL] pgAdmin4 needs information of v10 SCRAM authentication

2017-04-21 Thread Michael Paquier
On Fri, Apr 21, 2017 at 3:43 PM, Akshay Joshi wrote: >Thanks Michael, will check this. One thing I forgot to mention... Both StoredKey and ServerKey are now encoded in hex, but there is still an open item related to the handling of psql's \password on which I

Re: [pgadmin-hackers] [GENERAL] pgAdmin4 needs information of v10 SCRAM authentication

2017-04-21 Thread Akshay Joshi
On Fri, Apr 21, 2017 at 12:07 PM, Michael Paquier wrote: > On Fri, Apr 21, 2017 at 3:27 PM, Akshay Joshi > wrote: > > from passlib.hash import scram > > hash = scram.encrypt(data['newPassword']) -- This function provide > password for

Re: [pgadmin-hackers] [GENERAL] pgAdmin4 needs information of v10 SCRAM authentication

2017-04-21 Thread Michael Paquier
On Fri, Apr 21, 2017 at 3:27 PM, Akshay Joshi wrote: > from passlib.hash import scram > hash = scram.encrypt(data['newPassword']) -- This function provide password > for all the supported digest like [md5, sha-1, sha-256, sha-512]. Didn't work > I have tried with

[pgadmin-hackers] pgAdmin4 needs information of v10 SCRAM authentication

2017-04-21 Thread Akshay Joshi
Hi All We are into development phase where we are trying to incorporate the v10 changes into pgAdmin4. v10 added support for the SCRAM authentication into database server, so pgAdmin4 needs to incorporate that feature(for 'Change Password'). Now problem I am facing is, unable to find correct set