search

philbertphotos / osticket-multildap-auth-plugin

Plugin for OS Ticket that allows for authentication with multiple domains.
GNU General Public License v3.0
28 stars 17 forks source link

LDAP and PHP Errors #65

Closed siilitman closed 1 year ago

siilitman commented 1 year ago

Upgraded my OSTicket from 1.15.8 to 1.17.2 and flipped the php to 8.1

Initially, I couldn't access the site due to a php error with a miss match between getconfig($id) and getconfig() My eventual fix for that was to edit ./scp/sync_mldap.php line 116 and remove the $id The site started working and I was able to login.

I'm now facing the issue of any LDAP activity throws an error in the logs and often doesn't work or display results I'm seeing this in the apache error logs Uncaught TypeError: trim(): Argument #1 ($string) must be of type string, array given in /var/www/html/help/include/class.format.php:148\nStack trace:\n#0 Uncaught Error: Class "LDAPMultiAuthentication" not found in /var/www/html/help/scp/sync_mldap.php:2\nStack trace:\n#0 {main}\n

What should I be looking at to narrow this down a bit?

philbertphotos commented 1 year ago

I have not tested it in 1.17.2 I will do that today and get back to you. Did you update the plugin to the latest version and delete the sync_mldap.php first so it would be forced to update.

philbertphotos commented 1 year ago

As you can see line 116 does not have any $id so I am sure its a old copy of the plugin and it should work with 1.17.2 with no issues since the changes wont affect how the plugin works. image

siilitman commented 1 year ago

I did upgrade the plugin as I upgraded the OSTicket but I didn’t know about deleting the sync_mldap.php!

philbertphotos commented 1 year ago

@siilitman its a stupid bug that affects some systems. I have checks and balances but some times it wont allow the PHP to delete and copy the new file like it should.

philbertphotos commented 1 year ago

FIXED