fix(configure): mask gateway token prompts · openclaw/ope...

Recent Commits to openclaw:main

anurag-bg-ne · 2026-06-14 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@ import type { RuntimeEnv } from "../runtime.js";`
`5`	`5`
`6`	`6`	`const mocks = vi.hoisted(() => ({`
`7`	`7`	`text: vi.fn(),`
	`8`	`+password: vi.fn(),`
`8`	`9`	`select: vi.fn(),`
`9`	`10`	`confirm: vi.fn(),`
`10`	`11`	`resolveGatewayPort: vi.fn(),`
`@@ -24,6 +25,7 @@ vi.mock("../config/config.js", async (importActual) => {`
`24`	`25`
`25`	`26`	`vi.mock("./configure.shared.js", () => ({`
`26`	`27`	`text: mocks.text,`
	`28`	`+password: mocks.password,`
`27`	`29`	`select: mocks.select,`
`28`	`30`	`confirm: mocks.confirm,`
`29`	`31`	`}));`
`@@ -77,6 +79,7 @@ async function runGatewayPrompt(params: {`
`77`	`79`	`return input.initialValue ?? input.options[0]?.value;`
`78`	`80`	`});`
`79`	`81`	`mocks.text.mockImplementation(async () => params.textQueue.shift());`
	`82`	`+mocks.password.mockImplementation(async () => params.textQueue.shift());`
`80`	`83`	`mocks.randomToken.mockReturnValue(params.randomToken ?? "generated-token");`
`81`	`84`	`mocks.confirm.mockResolvedValue(params.confirmResult ?? true);`
`82`	`85`	`mocks.buildGatewayAuthConfig.mockImplementation((input) =>`
`@@ -115,6 +118,9 @@ describe("promptGatewayConfig", () => {`
`115`	`118`	`authConfigFactory: ({ mode, token, password }) => ({ mode, token, password }),`
`116`	`119`	`});`
`117`	`120`	`expect(result.token).toBe("generated-token");`
	`121`	`+expect(mocks.password).toHaveBeenCalledWith(`
	`122`	`+expect.objectContaining({ message: "Gateway token (blank to generate)" }),`
	`123`	`+);`
`118`	`124`	`});`
`119`	`125`
`120`	`126`	`it("does not set password to literal 'undefined' when prompt returns undefined", async () => {`
`@@ -126,6 +132,12 @@ describe("promptGatewayConfig", () => {`
`126`	`132`	`});`
`127`	`133`	`expect(call.password).not.toBe("undefined");`
`128`	`134`	`expect(call.password).toBe("");`
	`135`	`+expect(mocks.password).toHaveBeenCalledWith(`
	`136`	`+expect.objectContaining({`
	`137`	`+message: "Gateway password",`
	`138`	`+validate: expect.any(Function),`
	`139`	`+}),`
	`140`	`+);`
`129`	`141`	`});`
`130`	`142`
`131`	`143`	`it("prompts for trusted-proxy configuration when trusted-proxy mode selected", async () => {`

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ import { findTailscaleBinary } from "../infra/tailscale.js";`
`21`	`21`	`import type { RuntimeEnv } from "../runtime.js";`
`22`	`22`	`import { resolveDefaultSecretProviderAlias } from "../secrets/ref-contract.js";`
`23`	`23`	`import { buildGatewayAuthConfig } from "./configure.gateway-auth.js";`
`24`		`-import { confirm, select, text } from "./configure.shared.js";`
	`24`	`+import { confirm, password, select, text } from "./configure.shared.js";`
`25`	`25`	`import {`
`26`	`26`	`guardCancel,`
`27`	`27`	`normalizeGatewayTokenInput,`
`@@ -233,9 +233,8 @@ export async function promptGatewayConfig(`
`233`	`233`	note(`Validated ${envVarName}. OpenClaw will store a token SecretRef.`, "Gateway token");
`234`	`234`	`} else {`
`235`	`235`	`const tokenInput = guardCancel(`
`236`		`-await text({`
	`236`	`+await password({`
`237`	`237`	`message: "Gateway token (blank to generate)",`
`238`		`-initialValue: randomToken(),`
`239`	`238`	`}),`
`240`	`239`	`runtime,`
`241`	`240`	`);`
`@@ -245,14 +244,14 @@ export async function promptGatewayConfig(`
`245`	`244`	`}`
`246`	`245`
`247`	`246`	`if (authMode === "password") {`
`248`		`-const password = guardCancel(`
`249`		`-await text({`
	`247`	`+const passwordInput = guardCancel(`
	`248`	`+await password({`
`250`	`249`	`message: "Gateway password",`
`251`	`250`	`validate: validateGatewayPasswordInput,`
`252`	`251`	`}),`
`253`	`252`	`runtime,`
`254`	`253`	`);`
`255`		`-gatewayPassword = normalizeOptionalString(password) ?? "";`
	`254`	`+gatewayPassword = normalizeOptionalString(passwordInput) ?? "";`
`256`	`255`	`}`
`257`	`256`
`258`	`257`	`if (authMode === "trusted-proxy") {`

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@ import {`
`3`	`3`	`confirm as clackConfirm,`
`4`	`4`	`intro as clackIntro,`
`5`	`5`	`outro as clackOutro,`
	`6`	`+password as clackPassword,`
`6`	`7`	`select as clackSelect,`
`7`	`8`	`text as clackText,`
`8`	`9`	`} from "@clack/prompts";`
`@@ -104,3 +105,9 @@ export const select = <T>(params: Parameters<typeof clackSelect<T>>[0]) =>`
`104`	`105`	`opt.hint === undefined ? opt : { ...opt, hint: stylePromptHint(opt.hint) },`
`105`	`106`	`),`
`106`	`107`	`});`
	`108`	`+/** Styled password prompt wrapper. */`
	`109`	`+export const password = (params: Parameters<typeof clackPassword>[0]) =>`
	`110`	`+clackPassword({`
	`111`	`+ ...params,`
	`112`	`+message: stylePromptMessage(params.message),`
	`113`	`+});`

Original file line number	Diff line number	Diff line change
`@@ -10,6 +10,7 @@ const mocks = vi.hoisted(() => {`
`10`	`10`	`clackSelect: vi.fn(),`
`11`	`11`	`clackText: vi.fn(),`
`12`	`12`	`clackConfirm: vi.fn(),`
	`13`	`+clackPassword: vi.fn(),`
`13`	`14`	`resolveSearchProviderOptions: vi.fn(),`
`14`	`15`	`resolvePluginContributionOwners: vi.fn(),`
`15`	`16`	`setupSearch: vi.fn(),`
`@@ -46,6 +47,7 @@ vi.mock("@clack/prompts", () => ({`
`46`	`47`	`select: mocks.clackSelect,`
`47`	`48`	`text: mocks.clackText,`
`48`	`49`	`confirm: mocks.clackConfirm,`
	`50`	`+password: mocks.clackPassword,`
`49`	`51`	`}));`
`50`	`52`
`51`	`53`	`vi.mock("../config/config.js", () => ({`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main