Skip to content

Directory Sync : authentik integration #2347

Open
manab-pr wants to merge 27 commits into
theopenlane:mainfrom
manab-pr:feature/authentik
Open

Directory Sync : authentik integration #2347
manab-pr wants to merge 27 commits into
theopenlane:mainfrom
manab-pr:feature/authentik

Conversation

@manab-pr
Copy link
Copy Markdown
Contributor

@manab-pr manab-pr commented May 3, 2026

closes : Directory Sync #2342

Directory Sync Authentik implemenation

@manab-pr manab-pr requested a review from a team as a code owner May 3, 2026 17:07
@github-actions github-actions Bot added the enhancement New feature or request label May 3, 2026
@manab-pr manab-pr marked this pull request as draft May 3, 2026 17:08
@manab-pr manab-pr marked this pull request as ready for review May 8, 2026 03:04
@manab-pr
Copy link
Copy Markdown
Contributor Author

manab-pr commented May 8, 2026

@golanglemonade there are two things though : first , check the end points , i have put them in client.go , all the other integrations are using sdk so i couldn't find a reference where exactly should i put them , and second is : once you verify it and everything looks good , i'll write the documentation .

Comment thread internal/integrations/definitions/authentik/client.go Outdated
Comment thread internal/integrations/definitions/authentik/mappings.go
Comment thread internal/integrations/definitions/authentik/mappings.go Outdated
Comment thread internal/integrations/definitions/authentik/mappings.go Outdated
Comment thread internal/integrations/definitions/authentik/mappings.go
Comment thread internal/integrations/definitions/authentik/client.go Outdated
Comment thread internal/integrations/definitions/authentik/operation_directory_sync.go Outdated
Comment thread internal/integrations/definitions/authentik/builder.go
@manab-pr manab-pr requested a review from golanglemonade May 10, 2026 07:22
Comment thread docker/docker-compose-authentik.yml
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
8.2% Coverage on New Code (required ≥ 20%)

See analysis details on SonarQube Cloud

@manab-pr manab-pr requested a review from golanglemonade May 15, 2026 13:49
services:
postgresql:
env_file:
- authentik.env
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this is a dev setup, we can provide this. Add it to docker/configs/authentik and add the required env vars, e.g.:

AUTHENTIK_BOOTSTRAP_PASSWORD=akadmin123

That way it will just work without additional user setup to test this.

Comment on lines +6 to +7
POSTGRES_DB: ${PG_DB:-authentik}
POSTGRES_PASSWORD: ${PG_PASS:?database password required}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here, this is a dev env so we can set this up.

Alternatively, can we use the same postrgres container is is already coming up as part of the main stack and just create the db in there?

AUTHENTIK_POSTGRESQL__NAME: ${PG_DB:-authentik}
AUTHENTIK_POSTGRESQL__PASSWORD: ${PG_PASS}
AUTHENTIK_POSTGRESQL__USER: ${PG_USER:-authentik}
AUTHENTIK_SECRET_KEY: ${AUTHENTIK_SECRET_KEY:?secret key required}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should also be set and not required

Comment thread internal/integrations/definitions/authentik/installation.go Outdated
Comment thread internal/integrations/definitions/authentik/installation.go Outdated
Comment thread internal/integrations/definitions/authentik/mappings.go Outdated
Comment thread internal/integrations/definitions/authentik/mappings.go Outdated
Comment thread internal/integrations/definitions/authentik/types.go Outdated
Comment thread internal/integrations/definitions/authentik/operation_health.go Outdated
Copy link
Copy Markdown
Member

@golanglemonade golanglemonade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work, @manab-pr. this is great.

I tested locally and all seems to work:

image image image

Just a few final comments. Once we get docs in we can PR this again to set to active + visible. 🚀

manab-pr and others added 4 commits May 16, 2026 00:06
…y_sync.go

Co-authored-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: MANAB   PRATIM MAITY <79935184+manab-pr@users.noreply.github.com>
…y_sync.go

Co-authored-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: MANAB   PRATIM MAITY <79935184+manab-pr@users.noreply.github.com>
Co-authored-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: MANAB   PRATIM MAITY <79935184+manab-pr@users.noreply.github.com>
Co-authored-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: MANAB   PRATIM MAITY <79935184+manab-pr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request local-development run-ci used for forks for the CI to run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants