-
Notifications
You must be signed in to change notification settings - Fork 142
feat: add Zitadel host integration #1021
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
base: main
Are you sure you want to change the base?
Conversation
|
@dotnet-policy-service agree |
aaronpowell
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mentioned a problem with hostnames, can you tag the line(s) where that's surfacing?
Currently that would be https://github.com/CommunityToolkit/Aspire/pull/1021/files/677452ebe38d7be79caf11268057182f5594136d#diff-427aa76f92a729429b7b134c3be02f0a722e8ab12d85a50a2bc822d9819cda33R35 Zitadel requires a "stable" external hostname or it'll return a 404 if the This means that if I configure the external hostname as Currently I'm setting it to |
b84f375 to
10c9a28
Compare
|
rebased changes on |
Closes #1015
Adds host support for Zitadel to Aspire.
PR Checklist
Other information
Currently in draft since I haven't figured out everything.
<name>but if you open from the dashboard it'slocalhost:<port>. Currently it uses a semi-hardcoded<name>.dev.localhostas a middle ground.I played around with something similar to Keycloak's
WithRealmImportto "seed" the instance but AFAIK Zitadel doesn't support such things.I need to update some of the docs that are missing and write the tests, not sure if the example I've included is sufficient for now