string + email + nullish fails when empty string

[mmahalwy]

const emailSchema = z.string().email().nullish();

emailSchema.parse("")

fails with invalid email. This should not be the case due to nullish no?

[JacobWeisenburger]

An empty string is not considered a valid email based on emailRegex. See source code below:

zod/src/types.ts

Lines 443 to 445 in 004eefe

	// const emailRegex = /^((([a-z]|\d|[!#\$%&'\*\+\-\/=\?\^_`{\|}~]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])+(\.([a-z]|\d|[!#\$%&'\*\+\-\/=\?\^_`{\|}~]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])+)*)|((\x22)((((\x20|\x09)*(\x0d\x0a))?(\x20|\x09)+)?(([\x01-\x08\x0b\x0c\x0e-\x1f\x7f]|\x21|[\x23-\x5b]|[\x5d-\x7e]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(\\([\x01-\x09\x0b\x0c\x0d-\x7f]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF]))))*(((\x20|\x09)*(\x0d\x0a))?(\x20|\x09)+)?(\x22)))@((([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])([a-z]|\d|-|\.|_|~|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])*([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])))\.)+(([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])([a-z]|\d|-|\.|_|~|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])*([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])))$/i;
	// eslint-disable-next-line
	const emailRegex = /^(([^<>()[\]\.,;:\s@\"]+(\.[^<>()[\]\.,;:\s@\"]+)*)|(\".+\"))@(([^<>()[\]\.,;:\s@\"]+\.)+[^<>()[\]\.,;:\s@\"]{2,})$/i;

zod/src/types.ts

Lines 487 to 495 in 004eefe

	} else if (check.kind === "email") {
	if (!emailRegex.test(ctx.data)) {
	addIssueToContext(ctx, {
	validation: "email",
	code: ZodIssueCode.invalid_string,
	message: check.message,
	});
	status.dirty();
	}

---

Nullish doesn't affect this, because "" !== null

[mmahalwy]

@JacobWeisenburger makes sense. I feel like either: 1. we have something that allows empty string for string() or 2. allow nullish to also cover empty string. thoughts @colinhacks ?

[michaelhays]

As a workaround, you can use preprocess to convert the empty string to undefined:

const emailSchema = z.preprocess(
  (value) => (value === '' ? undefined : value),
  z.string().email().nullish(),
)