Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autodiscovery LDAP with service restart #96

Merged
merged 3 commits into from
Nov 29, 2024

Conversation

stephdl
Copy link
Contributor

@stephdl stephdl commented Nov 28, 2024

Implement LDAP autodiscovery functionality and ensure the web application restarts to apply the new configuration. This includes updates to the service files and the addition of a new script for LDAP discovery.

NethServer/dev#7103
NethServer/dev#7182

what is tested and workable

  1. new module installation
  2. migration from NS7
  3. new mail server from the settings page
  4. change of the ldap account of the mail server, propagated to webtop

imageroot/bin/discover-ldap Dismissed Show dismissed Hide dismissed
@stephdl stephdl force-pushed the sdl-7103-ldap-autodiscovery branch from eed7f76 to 68f8812 Compare November 28, 2024 14:56
@Amygos
Copy link
Member

Amygos commented Nov 28, 2024

Please. create a dedicated issue for Webtop, so the testing/QA phase can be tracked cleanly.

@gsanchietti
Copy link
Member

IMO the issue above is clear enough: it already describe the problem.
The patch is to correct that issue.

@Amygos
Copy link
Member

Amygos commented Nov 28, 2024

IMO the issue above is clear enough: it already describe the problem. The patch is to correct that issue.

Do you mean to this issue? NethServer/dev#7182

@stephdl
Copy link
Contributor Author

stephdl commented Nov 28, 2024

Do you mean to this issue? NethServer/dev#7182

the main issue is NethServer/dev#7103

@gsanchietti gsanchietti merged commit e1bd7f3 into main Nov 29, 2024
8 checks passed
@gsanchietti gsanchietti deleted the sdl-7103-ldap-autodiscovery branch November 29, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants