feat: docker compose for full hosting set up#2569
Conversation
|
|
||
| // Canonical GitHub App webhook endpoint. | ||
| r.POST("/github/webhook", diggerController.GithubAppWebHook) | ||
| // Legacy webhook path kept for backward compatibility. |
There was a problem hiding this comment.
We could leave this as is, but I figured we might want to standardize on /github/.
| // - "" or "/" -> "" | ||
| // - "orchestrator" -> "/orchestrator" | ||
| // - "/orchestrator/" -> "/orchestrator" | ||
| func NormalizePublicPathPrefix(raw string) string { |
There was a problem hiding this comment.
This isn't strictly necessary, but helps fix common issues people may encounter with leading and trailing slashes.
|
|
||
| await saveSession({ accessToken, refreshToken, user, impersonator }); | ||
|
|
||
| // Ensure the signed-in WorkOS organization exists in dependent services. |
There was a problem hiding this comment.
We could also leave this out for now and add in a separate PR cleaning up the callback/webhook logic
| } | ||
| | Response | ||
|
|
||
| export async function requireUiAuth(request: Request): Promise<AuthResult> { |
There was a problem hiding this comment.
If we wanted to keep things simpler, we could just return a 401.
Or this could be moved to a separate PR to keep this one more focused.
| return strings.EqualFold(vcsSpec.VcsType, "noop"), nil | ||
| } | ||
|
|
||
| func ProjectDriftStateMachineApply(project models.Project, tfplan string, resourcesCreated uint, resourcesUpdated uint, resourcesDeleted uint) error { |
There was a problem hiding this comment.
This duplicates the logic in
digger/drift/controllers/ci_jobs.go
Line 119 in 54070bc
1 participant
🧠 Ai UsageDetails (if applicable):
IMPORTANT: Please disclose any usage of ai tooling while making this change. If you did not use any AI write "NA" below
Initial implementation pass performed via opencode + codex. Review + testing + refactoring performed both manually and via opencode.