mirror of
https://github.com/andwati/dokploy.git
synced 2026-10-18 03:02:56 +00:00
Merge pull request #4933 from CyrilBIENNE/fix/preview-deployment-github-credentials
fix(preview-deployment): refetch github provider before authenticating
(cherry picked from commit a058d1f5c7)
This commit is contained in:
committed by
Mauricio Siu
parent
cbe78950cc
commit
0f0102ff2d
@@ -17,7 +17,7 @@ import { manageDomain } from "../utils/traefik/domain";
|
|||||||
import { findApplicationById } from "./application";
|
import { findApplicationById } from "./application";
|
||||||
import { removeDeploymentsByPreviewDeploymentId } from "./deployment";
|
import { removeDeploymentsByPreviewDeploymentId } from "./deployment";
|
||||||
import { createDomain } from "./domain";
|
import { createDomain } from "./domain";
|
||||||
import { type Github, getIssueComment } from "./github";
|
import { findGithubById, getIssueComment } from "./github";
|
||||||
import { getWebServerSettings } from "./web-server-settings";
|
import { getWebServerSettings } from "./web-server-settings";
|
||||||
|
|
||||||
export type PreviewDeployment = typeof previewDeployments.$inferSelect;
|
export type PreviewDeployment = typeof previewDeployments.$inferSelect;
|
||||||
@@ -142,7 +142,17 @@ export const createPreviewDeployment = async (
|
|||||||
org?.ownerId || "",
|
org?.ownerId || "",
|
||||||
);
|
);
|
||||||
|
|
||||||
const octokit = authGithub(application?.github as Github);
|
if (!application.githubId) {
|
||||||
|
throw new TRPCError({
|
||||||
|
code: "NOT_FOUND",
|
||||||
|
message: "Github Account not configured correctly",
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
// `findApplicationById` redacts `githubPrivateKey` from the `github`
|
||||||
|
// relation, so the provider must be refetched to authenticate.
|
||||||
|
const githubProvider = await findGithubById(application.githubId);
|
||||||
|
const octokit = authGithub(githubProvider);
|
||||||
|
|
||||||
const runningComment = getIssueComment(
|
const runningComment = getIssueComment(
|
||||||
application.name,
|
application.name,
|
||||||
|
|||||||
Reference in New Issue
Block a user