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.

Sign up for GitHub

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

Jump to bottom

feat(ldap-sync) #106

Draft
wants to merge 40 commits into
base: main
Choose a base branch
from
Draft

feat(ldap-sync) #106

wants to merge 40 commits into from

Conversation

kmendell
Copy link

PoC of users from ldap so far, Drafting this PR for code review first, and we can continue on from there.

kmendell and others added 28 commits January 11, 2025 11:42
@kmendell
feat: ldap sync begin work
ad4356b
@kmendell
feat(ldap-sync): add go-ldap
965191c
@kmendell
feat(ldap-sync): add env varibles for ldap
b6c4c0d
@kmendell
feat(ldap-sync): add env varibles for ldap
9eccfa3
@kmendell
feat(ldap-sync): updated .env.example with ldap config
324d83b
@kmendell
feat(ldap-sync): added basic ldapInit() function
a4f281e
@kmendell
feat(ldap-sync): added query function
577de76
@kmendell
feat(ldap-sync): added a few error checks for my testing
6fda6f4
@kmendell
feat(ldap-sync): updated .env.example
c019676
@kmendell
feat(ldap-sync): updated readme with the ldap env vars so far
0dda254
@kmendell
feat(ldap-sync): added a comment in main.go
839ff71
@kmendell
feat(ldap-sync): fixed main.go to restore normal functionality
09f4c17
@kmendell
feat(ldap-sync): Merge Result into UserObject model
72b269e
@kmendell
feat(ldap-sync): added new env var for user attribute, need better lo…
16dceef 
…gic on how to use that though
@kmendell
feat(ldap-sync): fixed the issue with username attributes, will map d…
8d480dc 
…irectly to the model.User struct now
@kmendell
feat(ldap-sync): removed unneeded ldap_types.go
710b793
@kmendell
feat(ldap-sync): restored main.go functionality
1e3b050
@kmendell
feat(ldap-sync): add initial group logic
0d552a1
@kmendell
feat(ldap-sync): added friendly name for group model
224c2ab
@kmendell
feat(ldap-sync): updated readme with env vars
ba6355a
@kmendell
feat(ldap-sync): updated .env.examples
a7d1e79
@kmendell
feat(ldap-sync): added skel for ldap_service and ldap_job
085a6f5
@kmendell
feat(ldap-sync): first draft of the ldap_service.go
7daf16d
@kmendell
feat(ldap-sync): fix import issue
7ddcb62
@kmendell
feat(ldap-sync): fixed return statment for else clause
b4818d2
@kmendell
feat(ldap-sync): fixed userService to use depend injection
27d441c
@kmendell
feat(ldap-sync): added ldapService to router bootstrap
7e546c8
@kmendell
Merge branch 'stonith404:main' into ldap-sync
7025e91
@kmendell
Copy link
Author

Current State of this PR as of commit: 7025e91 is users only in the ldap service, waiting for testing to confirm functionality, and then we can continue on with groups.

@kmendell kmendell changed the title WIP: feat(ldap-sync) feat(ldap-sync) Jan 15, 2025
@kmendell
Copy link
Author

Functionality for Sync is all working.
Few Issues/Notes:

  • If a user is removed from LDAP the user does not get deleted in pocket-id
    • I compared that to other services and it seems to be the same the users don't get deleted, that is more of SCIM territory.
  • We need to create the Postgres migration for the LDAP ID value in the DB.
  • Complete the Frontend UI Tweaks to not allow editing of LDAP Users or LDAP Groups

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kmendell @stonith404