From 7b451bbf6e6337507f4627b5a845f5bd96ab4f7b Mon Sep 17 00:00:00 2001 From: xiaojunnuo Date: Tue, 13 Aug 2024 20:30:42 +0800 Subject: [PATCH] =?UTF-8?q?perf:=20=E4=BC=98=E5=8C=96=E6=88=90=E5=8A=9F?= =?UTF-8?q?=E5=90=8E=E8=B7=B3=E8=BF=87=E7=9A=84=E6=8F=90=E7=A4=BA?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- packages/core/pipeline/src/plugin/api.ts | 7 ++- .../src/plugin/cert-plugin/base.ts | 7 +-- .../component/output-selector/index.vue | 4 +- .../pipeline/component/step-form/index.vue | 47 +++++++++++-------- packages/ui/certd-server/.env.pgdev.yaml | 3 -- packages/ui/certd-server/.env.preview.yaml | 4 +- packages/ui/certd-server/.env.production.yaml | 10 +++- .../certd-server/src/config/config.default.ts | 3 ++ .../certd-server/src/config/config.preview.ts | 21 --------- .../src/config/config.production.ts | 21 --------- .../certd-server/src/config/config.syncdb.ts | 11 ----- .../src/config/config.unittest.ts | 7 --- packages/ui/certd-server/src/configuration.ts | 10 +--- .../authority/controller/user-controller.ts | 10 +--- .../modules/authority/service/user-service.ts | 13 +++++ .../src/modules/auto/auto-register-cron.ts | 6 +-- .../pipeline/service/history-service.ts | 9 ++-- .../pipeline/service/pipeline-service.ts | 4 +- .../plugin-aliyun/access/aliyun-access.ts | 3 +- .../plugin/deploy-to-ack-ingress/index.ts | 12 ++--- .../plugin/deploy-to-cdn/index.ts | 30 ++---------- .../plugin/upload-to-aliyun/index.ts | 14 ++---- .../plugins/plugin-deploy-to-cdn.ts | 14 ++---- .../plugin-demo/plugins/plugin-test.ts | 41 ++++++++-------- .../plugin/host-shell-execute/index.ts | 9 +--- .../plugin/upload-to-host/index.ts | 10 +--- .../src/plugins/plugin-other/access/index.ts | 1 + .../plugins/plugin-other/access/k8s-access.ts | 19 ++++++++ .../plugin-tencent/access/dnspod-access.ts | 3 +- .../plugin/deploy-to-cdn/index.ts | 11 +---- .../plugin/deploy-to-clb/index.ts | 15 ++---- .../plugin/deploy-to-tke-ingress/index.ts | 14 ++---- .../plugin/upload-to-tencent/index.ts | 10 +--- 33 files changed, 151 insertions(+), 252 deletions(-) delete mode 100644 packages/ui/certd-server/src/config/config.preview.ts delete mode 100644 packages/ui/certd-server/src/config/config.production.ts delete mode 100644 packages/ui/certd-server/src/config/config.syncdb.ts delete mode 100644 packages/ui/certd-server/src/config/config.unittest.ts create mode 100644 packages/ui/certd-server/src/plugins/plugin-other/access/index.ts create mode 100644 packages/ui/certd-server/src/plugins/plugin-other/access/k8s-access.ts diff --git a/packages/core/pipeline/src/plugin/api.ts b/packages/core/pipeline/src/plugin/api.ts index 4f18a9685..4da1c0d4f 100644 --- a/packages/core/pipeline/src/plugin/api.ts +++ b/packages/core/pipeline/src/plugin/api.ts @@ -6,7 +6,7 @@ import { IAccessService } from "../access/index.js"; import { IEmailService } from "../service/index.js"; import { IContext } from "../core/index.js"; import { AxiosInstance } from "axios"; -import { logger } from "../utils/index.js"; +import { ILogger, logger } from "../utils/index.js"; export enum ContextScope { global, @@ -69,6 +69,9 @@ export type TaskInstanceContext = { export abstract class AbstractTaskPlugin implements ITaskPlugin { _result: TaskResult = { clearLastStatus: false, files: [], pipelineVars: {} }; ctx!: TaskInstanceContext; + logger!: ILogger; + accessService!: IAccessService; + clearLastStatus() { this._result.clearLastStatus = true; } @@ -79,6 +82,8 @@ export abstract class AbstractTaskPlugin implements ITaskPlugin { setCtx(ctx: TaskInstanceContext) { this.ctx = ctx; + this.logger = ctx.logger; + this.accessService = ctx.accessService; } randomFileId() { diff --git a/packages/plugins/plugin-cert/src/plugin/cert-plugin/base.ts b/packages/plugins/plugin-cert/src/plugin/cert-plugin/base.ts index 527b01888..984cf437c 100644 --- a/packages/plugins/plugin-cert/src/plugin/cert-plugin/base.ts +++ b/packages/plugins/plugin-cert/src/plugin/cert-plugin/base.ts @@ -1,7 +1,6 @@ -import { AbstractTaskPlugin, HttpClient, IAccessService, IContext, Step, TaskInput, TaskOutput } from "@certd/pipeline"; +import { AbstractTaskPlugin, HttpClient, IContext, Step, TaskInput, TaskOutput } from "@certd/pipeline"; import dayjs from "dayjs"; import type { CertInfo } from "./acme.js"; -import { Logger } from "log4js"; import { CertReader } from "./cert-reader.js"; import JSZip from "jszip"; @@ -91,9 +90,7 @@ export abstract class CertApplyBasePlugin extends AbstractTaskPlugin { // }) csrInfo!: string; - logger!: Logger; userContext!: IContext; - accessService!: IAccessService; http!: HttpClient; lastStatus!: Step; @@ -103,8 +100,6 @@ export abstract class CertApplyBasePlugin extends AbstractTaskPlugin { cert?: CertInfo; async onInstance() { - this.accessService = this.ctx.accessService; - this.logger = this.ctx.logger; this.userContext = this.ctx.userContext; this.http = this.ctx.http; this.lastStatus = this.ctx.lastStatus as Step; diff --git a/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/output-selector/index.vue b/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/output-selector/index.vue index 0e21fa1c7..d4230fdfd 100644 --- a/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/output-selector/index.vue +++ b/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/output-selector/index.vue @@ -23,7 +23,7 @@ export default { const currentStepIndex = inject("currentStepIndex") as Ref; const currentTask = inject("currentTask") as Ref; - const getPluginGroups = inject("getPluginGroups") as Ref; + const getPluginGroups = inject("getPluginGroups") as any; const pluginGroups = getPluginGroups(); function onCreate() { options.value = pluginGroups.getPreStepOutputOptions({ @@ -42,7 +42,7 @@ export default { watch( () => { - return pluginGroups.value.map; + return pluginGroups.value?.map; }, () => { onCreate(); diff --git a/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/step-form/index.vue b/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/step-form/index.vue index b4c82a9b9..8d0b45e7b 100644 --- a/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/step-form/index.vue +++ b/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/step-form/index.vue @@ -61,24 +61,7 @@ - +