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

Allow a separate domain name for auth #67

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

bickelj
Copy link
Collaborator

@bickelj bickelj commented Mar 2, 2023

Let the auth (keycloak) configuration live at a separate domain name from the web (PDC implementation) domain name. Separate certificates are now expected by the .env.example, compose.yml, and proxy.conf.example files. This change creates a logical separation of concerns but keeps the implementation on the same host for now. In future a change of (tangible) hosting for either backend service can be made with minimal disruption from a user perspective.

Issue #66 Deploy Keycloak at auth.pdco

Let the auth (keycloak) configuration live at a separate domain name
from the web (PDC implementation) domain name. Separate certificates
are now expected by the `.env.example`, `compose.yml`, and
`proxy.conf.example` files. This change creates a logical separation
of concerns but keeps the implementation on the same host for now. In
future a change of (tangible) hosting for either backend service can
be made with minimal disruption from a user perspective.

Issue #66 Deploy Keycloak at auth.pdco
@bickelj bickelj requested review from jasonaowen, reefdog and slifty March 2, 2023 17:18
Copy link
Contributor

@jasonaowen jasonaowen left a comment

Choose a reason for hiding this comment

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

I don't know how to test this, but these changes all make sense!

@bickelj
Copy link
Collaborator Author

bickelj commented Mar 3, 2023

@jasonaowen Thank you for your review!

@bickelj bickelj merged commit ccb4b78 into main Mar 3, 2023
@bickelj bickelj deleted the 66-deploy-keycloak-at-auth.pdco branch March 3, 2023 17:20
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