I was having problems to install LDAP integration module in my site, wich uses PostgreSQL, and I have found the problem. In ldapauth.install, line 51:
basedn TEXT(1000) NOT NULL DEFAULT '',
Text type is unlimited length, so I think the correct type you might want to use is character varying. Line 51 looks like that:
basedn CHARACTER VARYING(1000) NOT NULL DEFAULT '',
PD: Im not english, and this is my first bug report, sorry if something is wrong.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | ldap_integration-pgsql_0.patch | 1.1 KB | dexpl |
| #1 | ldapauth.install.txt | 5.71 KB | maxtorete |
Comments
Comment #1
maxtorete commentedAnother error, this lines are not present in PostgreSQL installation case:
ldapdata_binddn VARCHAR(255) NOT NULL default '',
ldapdata_bindpw VARCHAR(255) NOT NULL default '',
ldapdata_bindpw_clear VARCHAR(2) NOT NULL default '',
ldapdata_rwattrs TEXT NOT NULL default '',
ldapdata_roattrs TEXT NOT NULL default '',
ldapdata_mappings TEXT NOT NULL default '',
I attach my installation file.
Comment #2
thinkernel commentedI found the same problem. I installed the 5.x-1.2 on a drupal5.x+postgresql8.1.*+freebsd box.
Maybe just remove the '(1000)' behind the 'text' is fine. I give the sample below.
basedn text NOT NULL DEFAULT '',
Comment #3
dexpl commentedAlso found either incorrect "basedn" field definition and missing lines reported by Mr. Fernández. After appliyng a tiny patch (see attachment) installation was OK and operation seems to be OK too
Comment #4
scafmac commentedI believe the table columns are not created after the text(1000) error, because those statements are already in the install file, but thanks.
This is fixed and the new release will be updated shortly.
Comment #5
scafmac commentedMy mistake. You were correct those columns mustn't have been added to the Postgress section of the install. Fix in Head and in coming release.
Any Postgres feedback is much appreciated since I don't have a PG server to test against.
Cheers.
Comment #6
(not verified) commented