Yeah. You're right.

Dave,

Please find the patch to resolve the issue.
(This is valid for both REL-1_10 branch and trunk).

--
Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise Postgres Company<http://www.enterprisedb.com>


On Wed, May 19, 2010 at 10:03 PM, fatmijumar fatmijumar <
fatmiju...@gmail.com> wrote:

> Dear Sir:
>
> I have seen in the changelog for pgadmin 1.10.3 your contribution in this
> point (www.pgadmin.org)
>
> 2010-04-22 DP  1.10.3  Don't offer debugging options to non-superusers on
>
>
>                        Postgres Plus Advanced Server < 8.4 which doesn't
>                        have support for user debugging [Ashesh Vashi]
>
> I would like to inform you that I am afraid that this is not OK. We are
> running pgadmin 1.10.3 with postgresql 8.4 (not edb), and now we are forced
> to be superusers to debug in 8.4.0, which was not the case with 1.10.2. I
> would appreciate if you can check this, please.
>
> Thanks in advance.
>
> Regards.
>
>
Index: debugger/debugger.cpp
===================================================================
--- debugger/debugger.cpp	(revision 8382)
+++ debugger/debugger.cpp	(working copy)
@@ -115,7 +115,7 @@
         return false;
 
     // EnterpriseDB 8.3 or earlier does not support debugging by the non-superuser
-    if (!obj->GetServer()->GetSuperUser() && !obj->GetConnection()->EdbMinimumVersion(8, 4))
+    if (!obj->GetServer()->GetSuperUser() && !obj->GetConnection()->EdbMinimumVersion(8, 4) && obj->GetConnection()->GetIsEdb())
         return false;
 
     if (!obj->IsCollection())
-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Reply via email to