huh_ldap: sql syntax error

**heinermueller** · 12.10.2022, 20:34

Hallo allerseits, beim importieren via

Code:

vendor/bin/contao-console cache:clear && vendor/bin/contao-console huh_ldap:sync_persons -vv

kommt dieser SQL-Fehler:
---------------------

Code:

 [OK] Inserted new tl_user instance for LDAP person " (uid: bmxxsavi)".


20:13:33 CRITICAL  [console] Error thrown while running command "huh_ldap:sync_persons -vv". Message: "An exception occurred while executing 'INSERT INTO tl_user (ldapUidNumber,email,username,name,groups,admin,dateAdded,tstamp,password,language,backendTheme,uploader) VALUES ('10005','xxx.xxx@xxx.net','bmxxsavi','','a:0:{}','',1665598413,1665598413,'$argon2id$v=19$m=65536,t=4,p=1$zaY5/GanJ+/6fehjGxxxxxxxxxCYwIOW1qE2hNWXvc6scC44dKZFsXk','de','flexible','DropZone')':

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'groups,admin,dateAdded,tstamp,password,language,backendTheme,uploader) VALUES ('' at line 1" ["exception" => Doctrine\DBAL\Exception\SyntaxErrorException^ { …},"command" => "huh_ldap:sync_persons -vv","message" => """  An exception occurred while executing 'INSERT INTO tl_user (ldapUidNumber,email,username,name,groups,admin,dateAdded,tstamp,password,language,backendTheme,uploader) VALUES ('10005','xxx.xxx@xxx.net','bmxxsavi','','a:0:{}','',1665598413,1665598413,'$argon2id$v=19$m=65536,t=4,p=1$zaYxxxxxxJUStCYwIOW1qE2hNWXvc6scC44dKZFsXk','de','flexible','DropZone')':\n  \n  SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'groups,admin,dateAdded,tstamp,password,language,backendTheme,uploader) VALUES ('' at line 1  """]

In AbstractMySQLDriver.php line 98:

  [Doctrine\DBAL\Exception\SyntaxErrorException]
  An exception occurred while executing 'INSERT INTO tl_user (ldapUidNumber,email,username,name,groups,admin,dateAdded,tstamp,password,language,backendTheme,uploader) VALUES ('10005','x.xx
  osavi@xx.net','bmxxsavi','','a:0:{}','',1665598413,1665598413,'$argon2id$v=19$m=65536,t=4,p=1$zaY5/xxxxxxxymryRFAJUStCYwIOW1qE2hNWXvc6scC44dKZFsXk','de','flexible','DropZo
  ne')':

  SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'g
  roups,admin,dateAdded,tstamp,password,language,backendTheme,uploader) VALUES ('' at line 1


Exception trace:
  at /var/www/xxx/website/contao/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractMySQLDriver.php:98
 Doctrine\DBAL\Driver\AbstractMySQLDriver->convertException() at /var/www/xxx/website/contao/vendor/doctrine/dbal/lib/Doctrine/DBAL/DBALException.php:182
 Doctrine\DBAL\DBALException::wrapException() at /var/www/xxx/website/contao/vendor/doctrine/dbal/lib/Doctrine/DBAL/DBALException.php:159
 Doctrine\DBAL\DBALException::driverExceptionDuringQuery() at /var/www/xxx/website/contao/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php:2226
 Doctrine\DBAL\Connection->handleExceptionDuringQuery() at /var/www/xxx/website/contao/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php:1313
 Doctrine\DBAL\Connection->executeQuery() at /var/www/xxx/website/contao/vendor/contao/core-bundle/src/Resources/contao/library/Contao/Database/Statement.php:274
 Contao\Database\Statement->query() at /var/www/xxx/website/contao/vendor/contao/core-bundle/src/Resources/contao/library/Contao/Database/Statement.php:248
 Contao\Database\Statement->execute() at /var/www/xxx/website/contao/vendor/heimrichhannot/contao-utils-bundle/src/Database/DatabaseUtil.php:908
 HeimrichHannot\UtilsBundle\Database\DatabaseUtil->insert() at /var/www/xxx/website/contao/vendor/heimrichhannot/contao-ldap-bundle/src/Util/LdapUtil.php:494
 HeimrichHannot\LdapBundle\Util\LdapUtil->syncPersons() at /var/www/xxx/website/contao/vendor/heimrichhannot/contao-ldap-bundle/src/Command/SyncPersonsCommand.php:112
 HeimrichHannot\LdapBundle\Command\SyncPersonsCommand->execute() at /var/www/xxx/website/contao/vendor/symfony/console/Command/Command.php:255
 Symfony\Component\Console\Command\Command->run() at /var/www/xxx/website/contao/vendor/symfony/console/Application.php:1027
 Symfony\Component\Console\Application->doRunCommand() at /var/www/xxx/website/contao/vendor/symfony/framework-bundle/Console/Application.php:97
 Symfony\Bundle\FrameworkBundle\Console\Application->doRunCommand() at /var/www/xxx/website/contao/vendor/symfony/console/Application.php:273
 Symfony\Component\Console\Application->doRun() at /var/www/xxx/website/contao/vendor/symfony/framework-bundle/Console/Application.php:83
 Symfony\Bundle\FrameworkBundle\Console\Application->doRun() at /var/www/xxx/website/contao/vendor/symfony/console/Application.php:149
 Symfony\Component\Console\Application->run() at /var/www/xxx/website/contao/vendor/contao/manager-bundle/bin/contao-console:38
 include() at /var/www/xxx/website/contao/vendor/bin/contao-console:115

---------------------

Das SQL-Statement ist an sich syntaktisch ok, wo kann da der Fehler liegen?

Beste Grüße & vielen Dank!
Heiner

**Spooky** · 12.10.2022, 20:58

Lösche die server_version Angabe aus der parameters.yml und baue den Symfony Application Cache danach neu auf.

**tab** · 12.10.2022, 21:00

Ich denke 'groups' ist ein reserviertes Wort in neuen MySQL Versionen.

**heinermueller** · 12.10.2022, 21:38

Danke für die schnelle Hilfe!

Zitat von Spooky

Lösche die server_version Angabe aus der parameters.yml und baue den Symfony Application Cache danach neu auf.

Code:

cat ./config/parameters.yml

# This file has been auto-generated during installation
parameters:
    database_host: localhost
    database_port: 3306
    database_user: con_xxxx
    database_password: iNxxxxxxxxxxxxxx43Afp1
    database_name: contao_xxx_db
    secret: 8f10axxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx1cb6a

... leider keine server_version - oder steht das wo anders?

**Spooky** · 12.10.2022, 21:48

Hm, prüfe mal deine config.yaml, wenn du eine hast.

**heinermueller** · 12.10.2022, 21:51

Zitat von tab

Ich denke 'groups' ist ein reserviertes Wort in neuen MySQL Versionen.

Tatsache .. ich habe mal 'groups' etwas gehackt und 'groupss' daraus gemacht und siehe da:

Code:

 An exception occurred while executing 'INSERT INTO tl_user (ldapUidNumber,email,username,name,groupss,admin,dateAdded,tstamp,password,language,backendTheme,uploader) VALUES ('10005','xxx.x@xxx.net','bmxxxxsavi','','a:0:{}','',1665607618,1665607618,'$argon2id$v=19$m=65536,t=4,p=1$xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxcfTig2aBeBbXf0','de','flexible','DropZone')':\n  \n  SQLSTATE[42S22]: Column not found: 1054 Unknown column 'groupss' in 'field list'  """]

In AbstractMySQLDriver.php line 79:

  [Doctrine\DBAL\Exception\InvalidFieldNameException]

Nice catch! Respekt! Und jetzt? Kann ich eine SQL-Version irgendwo erzwingen?

**heinermueller** · 12.10.2022, 21:59

Code:

$personData['groups']

->

Code:

$personData['`groups`']

funktioniert.

**Spooky** · 12.10.2022, 22:12

Vermutlich ein Bug im berühmt berüchtigten heimrichhannot/contao-utils-bundle ...

**koertho** · 13.10.2022, 08:29

Warum sollte ein in bestimmten MySQL-Versionen reservierter Feldname ein Bug im Utils-Bundle sein?
Zumal ja schon eine Lösung gepostet wurde. Habe daraus ein PR gemacht, kann gerne getestet werden. Habe leider weder ein Test-LDAP noch tieferes Verständnis des Bundle parat (habe es nicht entwickelt und der ursprüngliche Entwickler ist nicht mehr in der Firma).
https://github.com/heimrichhannot/co...-bundle/pull/3

**Spooky** · 13.10.2022, 08:39

Zitat von koertho

Warum sollte ein in bestimmten MySQL-Versionen reservierter Feldname ein Bug im Utils-Bundle sein?

Statt Doctrine\DBAL\Connection nutzt ihr ein DatabaseUtil aus dem Utils Bundle. Über Doctrine\DBAL\Connection sollte bei so einem insert Query der Feldname automatisch escaped werden (bin mir aber auch nicht zu 100% sicher).

**koertho** · 13.10.2022, 14:12

Das Util erzeugt aber auch nur nen Query aus dem Array und nutzt anschließen die Contao Datenbank-Klasse. Ist einfach ein Shorthand. Hätte man auch hier im Code machen könne, wäre aufs gleiche rausgekommen.
Unser Utils-Bundle hat sicher noch ein paar Leichen im Keller, aber ich denke wir haben schon ganz gut aufgeräumt...

**Spooky** · 13.10.2022, 14:14

Zitat von koertho

Das Util erzeugt aber auch nur nen Query aus dem Array und nutzt anschließen die Contao Datenbank-Klasse.

Es sollte aber Doctrine benutzt werden.

**koertho** · 18.10.2022, 07:37

Sicher. Es könnte vieles besser und anders gemacht sein. Aber leider verdienen wir damit kein Geld, nur unseren Code zu modernisieren. Er läuft und die Contao Datenbank-Klasse ist nicht deprecated und nutzt intern auch Doctrine. Nur weil es nicht den neusten Standards entspricht, muss es nicht schlecht sein.
So, aber eigentlich geht es hier nicht um unser Utils-Bundle, welches hier nicht schuld ist, sondern um LDAP. Dazu habe ich einen Fix gebaut und noch keine Rückmeldung erhalten.