Ticket #10319 (closed defect: fixed)

Opened 8 months ago

Last modified 8 months ago

Error with postgresql and Trac 1.0

Reported by: mcdevil@gmx.net Assigned to: sandinak
Priority: normal Component: DirectoryAuthPlugin
Severity: normal Keywords:
Cc: Trac Release: 1.0

Description (Last modified by rjollos)

I tried to upgraded from version 0.2.2 and have some problems

Installation:

  • since v0.3 a database upgrade is needed
  • this does only work if tracext.adauth.db.activedirectoryauthpluginsetup = enabled (this hint qould be good in the documentation)
  • the upgrade fails on postgresql with unknown type varcahar(32) -> typo: varchar(32)
  • the upgrade fails on postgresql with unknown type binary -> i tried: text

After this the installation and upgrade succeeds

But i've still an error: 

  File "/usr/local/lib/python2.6/dist-packages/TracActiveDirectoryAuth-0.3.2-py2.6.egg/tracext/adauth/auth.py", line 139, in check_password
    dn = self._get_user_dn(user)
  File "/usr/local/lib/python2.6/dist-packages/TracActiveDirectoryAuth-0.3.2-py2.6.egg/tracext/adauth/auth.py", line 271, in _get_user_dn
    if self.has_user(user):
  File "/usr/local/lib/python2.6/dist-packages/TracActiveDirectoryAuth-0.3.2-py2.6.egg/tracext/adauth/auth.py", line 129, in has_user
    users = self.get_users()
  File "/usr/local/lib/python2.6/dist-packages/TracActiveDirectoryAuth-0.3.2-py2.6.egg/tracext/adauth/auth.py", line 73, in get_users
    userinfo = self.expand_group_users(self.auth_group)
  File "/usr/local/lib/python2.6/dist-packages/TracActiveDirectoryAuth-0.3.2-py2.6.egg/tracext/adauth/auth.py", line 97, in expand_group_users
    g = self._ad_search(self.base_dn, ldap.SCOPE_SUBTREE, groupfilter, ['member'])
  File "/usr/local/lib/python2.6/dist-packages/TracActiveDirectoryAuth-0.3.2-py2.6.egg/tracext/adauth/auth.py", line 439, in _ad_search
    cur.execute("SELECT lut,data FROM ad_cache WHERE id='%s'" % key)
  File "/usr/local/lib/python2.6/dist-packages/Trac-1.0-py2.6.egg/trac/db/util.py", line 66, in execute
    return self.cursor.execute(sql)
InternalError: FEHLER:  aktuelle Transaktion wurde abgebrochen, Befehle werden bis zum Ende der Transaktion ignoriert

Any idea what's wrong?

Attachments

Change History

09/14/12 18:29:28 changed by anonymous

There is also a problem with the query definition:

_ad_search: 

cur.execute('SELECT lut,data FROM ad_cache WHERE id="%s"' % key)

postgresql does not like double quotes. I changed it to 

cur.execute("SELECT lut,data FROM ad_cache WHERE id='%s'" % key)

for now. I don't know if it's the correct way.

09/16/12 08:52:44 changed by rjollos

  • description changed.

09/16/12 09:28:19 changed by rjollos

I believe the correct database-agnostic fix for the issue in comment:1 is to utilize the db.quote(param) function. See t:TracDev/DatabaseApi#GuidelinesforSQLStatements and:

09/17/12 21:11:50 changed by rjollos

As a follow-up to comment:3, I should add that my suggestion was speculative, and I've not had occasion to use the db.quote function. I've been studying the Trac db API lately though, and it seems like the function is intended for this circumstance. I'll be interested to see what you find while implementing the fix.

(follow-up: ↓ 6 ) 09/18/12 14:33:40 changed by sandinak

  • status changed from new to closed.
  • resolution set to fixed.

Fixed the queries to match t:TracDev/DatabaseApi#GuidelinesforSQLStatements .. wish I had known that was there before .. ;-)

This fix will be in DirectoryAuthPlugin 0.5

(in reply to: ↑ 5 ) 09/18/12 15:04:59 changed by rjollos

Replying to sandinak:

Fixed the queries to match t:TracDev/DatabaseApi#GuidelinesforSQLStatements .. wish I had known that was there before .. ;-)

You are not alone. Failing to follow those guidelines is probably the most widespread problem with plugins.

Add/Change #10319 (Error with postgresql and Trac 1.0)




Change Properties
Action