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

Implement LDAP integration #286

Open
wants to merge 37 commits into
base: testing
Choose a base branch
from
Open

Conversation

looowizz
Copy link

@looowizz looowizz commented Feb 7, 2025

Problem

  • Vaultwarden did not automatically invite users created in Yunohost

Solution

Limitations

  • This package is only released for amd64 architecture, but I'm assuming it could be built manually to support arm. I have not included this, instead I've added a warning during installation that this only supports amd64 architecture

PR Status

  • Code finished and ready to be reviewed/tested
  • The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

looowizz and others added 30 commits November 17, 2024 00:08
- Add systemd configuration
- Apply ldap_config.toml
- Add yunohost service
- Add path to vaultwarden url in ldap config
- Add ldap configs to backup/restore
- Restart ldap service on fail (happened on startup for me)
- Add warning to user on install that only amd64 supported

Signed-off-by: loowiz <[email protected]>
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.

2 participants