From 40a949038290e21d9952591cdaa9efff678603fe Mon Sep 17 00:00:00 2001 From: Murtuza Zabuawala Date: Thu, 12 May 2016 13:22:13 +0530 Subject: [PATCH] Check for the connection status before executing the statement for checking the 'server in recovery' information. Thanks Khushboo for reporting the issue. --- .../browser/server_groups/servers/__init__.py | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/web/pgadmin/browser/server_groups/servers/__init__.py b/web/pgadmin/browser/server_groups/servers/__init__.py index 180807fb4..a79f1d47e 100644 --- a/web/pgadmin/browser/server_groups/servers/__init__.py +++ b/web/pgadmin/browser/server_groups/servers/__init__.py @@ -69,13 +69,15 @@ class ServerModule(sg.ServerGroupPluginModule): manager = driver.connection_manager(server.id) conn = manager.connection() connected = conn.connected() - - status, in_recovery = conn.execute_scalar(""" - SELECT CASE WHEN usesuper - THEN pg_is_in_recovery() - ELSE FALSE - END as inrecovery - FROM pg_user WHERE usename=current_user""") + if connected: + status, in_recovery = conn.execute_scalar(""" + SELECT CASE WHEN usesuper + THEN pg_is_in_recovery() + ELSE FALSE + END as inrecovery + FROM pg_user WHERE usename=current_user""") + else: + in_recovery = None yield self.generate_browser_node( "%d" % (server.id),