xperseguers / t3ext-ig_ldap_sso_auth

TYPO3 Extension ig_ldap_sso_auth. This extension provides LDAP and SSO support for TYPO3.
https://extensions.typo3.org/extension/ig_ldap_sso_auth
27 stars 68 forks source link

[BUGFIX] Adapt key length on DN for InnoDB tables #12

Closed xperseguers closed 6 years ago

xperseguers commented 6 years ago

According to https://dev.mysql.com/doc/refman/5.7/en/innodb-restrictions.html, an InnoDB table has an index key prefix limit of 767 bytes if configuration parameter innodb_large_prefix is disabled.

By constraining the DN key to 64 characters, we should still have a good discriminant index while preventing errors when trying to create the index in the Install Tool.

Resolves: #9

xperseguers commented 6 years ago

I'd like @tbrodard to confirm that this changes is fine. Thanks.

tbrodard commented 6 years ago

@xperseguers Tested on a TYPO3 7.6 instance: no error message while updating the db schema from the Install Tool

=> it's OK