Crivaledaz / Mattermost-LDAP

This module provides an external LDAP authentication in Mattermost for the Team Edition (free).
MIT License
359 stars 71 forks source link

Does it work with Mattermost 5.20.1 and PHP-FPM 7.3 ? #41

Closed jcharaoui closed 4 years ago

jcharaoui commented 4 years ago

I've been trying hard to deploy this on a Debian 10 server with Apache 2.4 and PHP-FPM 7.3, but so far I haven't had any luck. Being unfamiliar with oauth2 in general probably doesn't help. The LDAP part works fine, and I can see access tokens show up in the database, but it seems it's hitting problems when interacting with the Matthermost instance.

Looking at the OAuth2 source code, it seems like it was designed for the era of PHP 5.x, so it's unclear if I'm just wasting my time or perhaps someone else has had success making it work on a modern stack?

I also had a shot at oauth2_proxy with Azure as a provider but there again, I don't know if it's that's even a logical option.

b3 commented 4 years ago

Is it someting like #43?

Crivaledaz commented 4 years ago

Hi,

You are right, Mattermost-LDAP has been implemented for PHP 5.6. The module is based on the Oauth2 PHP implementation made by bshaffer (https://github.com/bshaffer/oauth2-server-php). In 2017, when I began the module, I took the last realease of his project, which was verion 1.9.0. I never took the time to update Mattermost-LDAP with new versions of bshaffer project. Maybe I will have the time during the lockdown ;)

Nevertheless, I have re-installed Mattermost and Mattermost-LDAP on my PC, and I can confirm that Mattermost-LDAP is working with Mattermost Team Edition 5.21.0 and PHP-FPM 7.3.

Here, my configuration in details :

Component Version
Fedora 31
PostgreSQL 11.7
Mattermost 5.21.0
Httpd 2.4.41
PHP-FPM 7.3.16

Let me know where you are stuck, what kind of error do you get ?

Regards,

Crivaledaz commented 4 years ago

Hi,

I finally found some time during the lockdown, and I have worked on a new Mattermost-LDAP version. So, I am glad to announce Mattermost-LDAP V2 is now in beta :)

This new version includes an update of bshaffer's Oauth server implementation. Mattermost-LDAP is now fully complaint with PHP 7, as you can see in the demo.

You can try this new version from the branch v2. For more informations about this version, see the changelog file

Please, let me know if this new version solves your issue.

Regards

jcharaoui commented 4 years ago

Thanks for working on this! As far as I'm concerned I've moved to to another project, but I'm quite certain this work will be useful for others. Feel free to close this issue.

Crivaledaz commented 4 years ago

I close this issue since PHP 7 is supported in Mattermost-LDAP V2.