From e43a96b19b2e7768220137e6444935f2ab39ae13 Mon Sep 17 00:00:00 2001
From: Lars Kanis <kanis@comcard.de>
Date: Tue, 24 May 2011 13:52:21 +0200
Subject: [PATCH] add support for postgres-pr

---
 README.txt => README.rdoc        |  0
 lib/pg_ldap_sync/application.rb  | 40 ++++++++++++++++++++++++--------
 test/fixtures/config-ldapdb.yaml |  6 ++++-
 3 files changed, 35 insertions(+), 11 deletions(-)
 rename README.txt => README.rdoc (100%)

diff --git a/README.txt b/README.rdoc
similarity index 100%
rename from README.txt
rename to README.rdoc
diff --git a/lib/pg_ldap_sync/application.rb b/lib/pg_ldap_sync/application.rb
index a29b44d..f32d8c0 100644
--- a/lib/pg_ldap_sync/application.rb
+++ b/lib/pg_ldap_sync/application.rb
@@ -5,9 +5,29 @@ require 'net/ldap'
 require 'optparse'
 require 'yaml'
 require 'logger'
-require 'pg'
 require 'kwalify'
 
+begin
+  require 'pg'
+rescue LoadError => e
+  begin
+    require 'postgres'
+    class PGconn
+      alias initialize_before_hash_change initialize
+      def initialize(*args)
+        arg = args.first
+        if args.length==1 && arg.kind_of?(Hash)
+          initialize_before_hash_change(arg[:host], arg[:port], nil, nil, arg[:dbname], arg[:user], arg[:password])
+        else
+          initialize_before_hash_change(*args)
+        end
+      end
+    end
+  rescue LoadError
+    raise e
+  end
+end
+
 require 'pg_ldap_sync'
 
 module PgLdapSync
@@ -111,8 +131,8 @@ class Application
     users = []
     res = @pgconn.exec "SELECT rolname FROM pg_roles WHERE #{pg_users_conf[:filter]}"
     res.each do |tuple|
-      user = PgRole.new tuple['rolname']
-      log.info{ "found pg-user: #{user.name}"}
+      user = PgRole.new tuple[0]
+      log.info{ "found pg-user: #{user.name.inspect}"}
       users << user
     end
     return users
@@ -124,10 +144,10 @@ class Application
     groups = []
     res = @pgconn.exec "SELECT rolname, oid FROM pg_roles WHERE #{pg_groups_conf[:filter]}"
     res.each do |tuple|
-      res2 = @pgconn.exec "SELECT pr.rolname FROM pg_auth_members pam JOIN pg_roles pr ON pr.oid=pam.member WHERE pam.roleid=$1", [{:value=>tuple['oid']}]
-      member_names = res2.field_values 'rolname'
-      group = PgRole.new tuple['rolname'], member_names
-      log.info{ "found pg-group: #{group.name}"}
+      res2 = @pgconn.exec "SELECT pr.rolname FROM pg_auth_members pam JOIN pg_roles pr ON pr.oid=pam.member WHERE pam.roleid=#{PGconn.escape(tuple[1])}"
+      member_names = res2.map{|row| row[0] }
+      group = PgRole.new tuple[0], member_names
+      log.info{ "found pg-group: #{group.name.inspect} with members: #{member_names.inspect}"}
       groups << group
     end
     return groups
@@ -186,10 +206,10 @@ class Application
     return roles
   end
 
-  def pg_exec(sql, params=nil)
-    log.info{ "SQL: #{sql}" + (params ? " params: #{params}" : '') }
+  def pg_exec(sql)
+    log.info{ "SQL: #{sql}" }
     unless self.test
-      @pgconn.exec sql, params
+      @pgconn.exec sql
     end
   end
 
diff --git a/test/fixtures/config-ldapdb.yaml b/test/fixtures/config-ldapdb.yaml
index a4353f0..940a2e2 100644
--- a/test/fixtures/config-ldapdb.yaml
+++ b/test/fixtures/config-ldapdb.yaml
@@ -15,8 +15,12 @@ ldap_groups:
   member_attribute: member
 
 pg_connection:
-  host: localhost
   dbname: postgres
+# needed for postgres-pr:
+#   host: localhost
+#   port: 54321
+#   user: username
+#   password: secret
 
 pg_users:
   filter: rolcanlogin AND NOT rolsuper