Skip to content

Add LLDAP automatic configuration for stalwart#6908

Draft
docjyJ wants to merge 1 commit intomainfrom
ench/noid/lldap-stalwart
Draft

Add LLDAP automatic configuration for stalwart#6908
docjyJ wants to merge 1 commit intomainfrom
ench/noid/lldap-stalwart

Conversation

@docjyJ
Copy link
Collaborator

@docjyJ docjyJ commented Sep 27, 2025

After automatic configuration for ClamAV, I would like to do the same for lldap.

I don't know if shared secrets work?

I think that if lldap is enabled, then the secret will be set, otherwise the secret will be empty. Do you agree?

Or else an enable_ldap variable would be required.

I would like your opinion on this feature.

@docjyJ docjyJ changed the title Add LLDAP automatic configuration Add LLDAP automatic configuration for stalwart Sep 27, 2025
@docjyJ docjyJ added 2. developing Work in progress enhancement New feature or request labels Sep 27, 2025
@docjyJ docjyJ added this to the next milestone Sep 27, 2025
@docjyJ docjyJ force-pushed the ench/noid/lldap-stalwart branch from 30ef4e6 to 121f5de Compare September 27, 2025 12:56
Signed-off-by: Jean-Yves <7360784+docjyJ@users.noreply.github.com>
@docjyJ docjyJ force-pushed the ench/noid/lldap-stalwart branch from 121f5de to 0c0870c Compare September 27, 2025 13:34
@docjyJ docjyJ self-assigned this Sep 27, 2025
@szaimen
Copy link
Collaborator

szaimen commented Sep 29, 2025

Hey :)

After automatic configuration for ClamAV, I would like to do the same for lldap.

Sounds good!

I don't know if shared secrets work?

Yes, should work but please add LLDAP_LDAP_USER_PASS to secrets of the stalwart.json

I think that if lldap is enabled, then the secret will be set, otherwise the secret will be empty. Do you agree?

No, the secret will be set all the time. So I guess we need a different way to detect if lldap is enabled. For example you could try to use dig to check if nextcloud-aio-lldap is there during the startup of stalwart...

Or else an enable_ldap variable would be required.

Yeah, I guess this would be the cleanest way...

I would like your opinion on this feature.

done :)

@szaimen
Copy link
Collaborator

szaimen commented Nov 24, 2025

@docjyJ any update here? :)

@docjyJ
Copy link
Collaborator Author

docjyJ commented Nov 25, 2025

No, I'm stuck. Because we have to save the LDAP configuration ID, and that's not possible at the moment.

I add the discussion label.

@docjyJ docjyJ marked this pull request as draft November 25, 2025 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress discussion Being discussed enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants