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

BED-4655: Add Authentik to docker-compose.dev.yml #900

Merged
merged 3 commits into from
Oct 16, 2024
Merged

Conversation

ALCooper12
Copy link
Contributor

@ALCooper12 ALCooper12 commented Oct 8, 2024

Description

  • Add authentik to docker-compose.dev.yml
  • Added profiles bh-sso as well as bh-sso-only for just support to quickly spin up authentik and bhe

Motivation and Context

  • Ticket: BED-4655

Why is this change required? What problem does it solve?

  • This adds authentik - https://github.com/goauthentik/authentik to the dev stack. It provides us with an IDP that supports a variety of SSO protocols so we can locally test and support OIDC / SAML auth flows.

How Has This Been Tested?

  • Run just bh-sso Note you have to go to http://authentik.localhost/if/flow/initial-setup/ to register an admin password before you can progress further

Screenshots (optional):

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

@ALCooper12 ALCooper12 self-assigned this Oct 8, 2024
@ALCooper12 ALCooper12 added api A pull request containing changes affecting the API code. tooling This updates developer tooling labels Oct 8, 2024
docker-compose.dev.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@ddlees ddlees left a comment

Choose a reason for hiding this comment

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

Please create a bh-api-sso declaration to launch BH with Authentik via the sso profile. The BH deployment for sso requires a slightly different configuration than the base deployment and we'll want to keep those separate and not interfere with normal development workflows.

.env.example Outdated Show resolved Hide resolved
@ALCooper12 ALCooper12 added the work in progress This pull request is a work in progress and should not be merged label Oct 11, 2024
docker-compose.dev.yml Outdated Show resolved Hide resolved
docker-compose.dev.yml Outdated Show resolved Hide resolved
docker-compose.dev.yml Outdated Show resolved Hide resolved
docker-compose.dev.yml Outdated Show resolved Hide resolved
docker-compose.dev.yml Outdated Show resolved Hide resolved
docker-compose.dev.yml Outdated Show resolved Hide resolved
docker-compose.dev.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@mistahj67 mistahj67 left a comment

Choose a reason for hiding this comment

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

I think the only thing needed left is the example vars don't match the docker yml, otherwise LGTM!

.env.example Outdated Show resolved Hide resolved
@ALCooper12 ALCooper12 removed the work in progress This pull request is a work in progress and should not be merged label Oct 16, 2024
@ALCooper12 ALCooper12 merged commit 3c12485 into main Oct 16, 2024
4 checks passed
@ALCooper12 ALCooper12 deleted the BED-4655 branch October 16, 2024 16:23
@github-actions github-actions bot locked and limited conversation to collaborators Oct 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api A pull request containing changes affecting the API code. tooling This updates developer tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants