Skip to content

Initiate docker compose for keycloak#3

Merged
hiento09 merged 1 commit into
mainfrom
feature/jan-1229
Aug 22, 2023
Merged

Initiate docker compose for keycloak#3
hiento09 merged 1 commit into
mainfrom
feature/jan-1229

Conversation

@hiento09
Copy link
Copy Markdown
Contributor

This PR contains the initiate docker compose for keycloak.
I will integrate the keycloak configuration with jan-web later.

@linear
Copy link
Copy Markdown

linear Bot commented Aug 22, 2023

JAN-1229 Deploy Keycloak in Docker Compose

  • Work with louis to deploy Keycloak in Docker Compose

Comment thread .gitignore
@@ -0,0 +1,2 @@
.vscode
.env No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you add a .env.sample? and add the secrets to 1pass? ty!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I will add an .env.sample file. All of these are for locally so I don't think we need to update them to 1password

Comment thread docker-compose.yml
environment:
KEYCLOAK_URL: http://keycloak:8080
KEYCLOAK_USER: ${KEYCLOAK_ADMIN-admin}
KEYCLOAK_PASSWORD: ${KEYCLOAK_ADMIN_PASSWORD-admin}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What's with the uppercase then lower case, and _ then -?
Is it intentional?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Example with KEYCLOAK_USER: ${KEYCLOAK_ADMIN-admin}:
KEYCLOAK_USER will get the value of $KEYCLOAK_ADMIN environment variable if exists, else the value admin will be set

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