self-service-password icon indicating copy to clipboard operation
self-service-password copied to clipboard

Improve code factorization

Open coudot opened this issue 3 years ago • 4 comments

We should at least create lib/ldap.inc.php to define some functions to connect to LDAP and search for an entry

coudot avatar Sep 01 '22 16:09 coudot

this will be part of https://github.com/ltb-project/white-pages/issues/119 current refactoring in #ltb-ldap branch.

artlog avatar Jan 05 '23 16:01 artlog

Because of some special burden, I needed to create some LDAPServer Classes.

How would you like to take on this issue, by providing code/pull requests or by general debating about it?

This this changes, I have to raise the question which php-version the code has to support. Based on the official support page, I would prefer writing code that fits to PHP 8.1.

What do you think? Should this be discussed in a separate issue ticket?

stevleibelt avatar Feb 14 '23 07:02 stevleibelt

Because of some special burden, I needed to create some LDAPServer Classes.

How would you like to take on this issue, by providing code/pull requests or by general debating about it?

This this changes, I have to raise the question which php-version the code has to support. Based on the official support page, I would prefer writing code that fits to PHP 8.1.

What do you think? Should this be discussed in a separate issue ticket?

Hello @stevleibelt sorry for the late reply.

We will use our common LTB lib: https://github.com/ltb-project/ltb-ldap

coudot avatar Aug 18 '23 16:08 coudot

Need to look at https://github.com/ltb-project/ltb-ldap/pull/7

coudot avatar Mar 07 '24 14:03 coudot

Done in https://github.com/ltb-project/ltb-ldap/pull/12

coudot avatar Apr 19 '24 13:04 coudot