jazzymc/homarr
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: app url not required (#2130)

Browse Source
This commit is contained in:
Meier Lukas
2025-01-26 21:25:57 +01:00
committed by GitHub
parent 2bb44e1666
commit b38de29a88
2 changed files with 16 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
apps/nextjs/src/app/[locale]/manage/apps/_form.tsx
View File

@@ -25,11 +25,11 @@ export const AppForm = (props: AppFormProps) => {
const t = useI18n(); const t = useI18n();
const form = useZodForm(validation.app.manage, { const form = useZodForm(validation.app.manage, {
initialValues: initialValues ?? { initialValues: {
name: "", name: initialValues?.name ?? "",
description: "", description: initialValues?.description ?? "",
iconUrl: "", iconUrl: initialValues?.iconUrl ?? "",
href: "", href: initialValues?.href ?? "",
}, },
}); });

14
packages/validation/src/app.ts
View File

@@ -1,13 +1,21 @@
import { z } from "zod"; import { z } from "zod";
const manageAppSchema = z.object({ const manageAppSchema = z.object({
name: z.string().min(1).max(64), name: z.string().trim().min(1).max(64),
description: z.string().max(512).nullable(), description: z
iconUrl: z.string().min(1), .string()
.trim()
.max(512)
.transform((value) => (value.length === 0 ? null : value))
.nullable(),
iconUrl: z.string().trim().min(1),
href: z href: z
.string() .string()
.trim()
.url() .url()
.regex(/^https?:\/\//) // Only allow http and https for security reasons (javascript: is not allowed) .regex(/^https?:\/\//) // Only allow http and https for security reasons (javascript: is not allowed)
.or(z.literal(""))
.transform((value) => (value.length === 0 ? null : value))
.nullable(), .nullable(),
}); });