Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Feb 4, 2026

This is an automated pull request to merge chas/update-domain-validation into dev.
It was created by the [Auto Pull Request] action.

@cursor
Copy link

cursor bot commented Feb 4, 2026

PR Summary

Low Risk
Low risk: changes are limited to input validation regexes for trust portal domains, broadening accepted values without altering verification or persistence logic.

Overview
Updates trust portal domain validation to support long TLDs by changing the domain regex from \.[A-Za-z]{2,6} to \.[A-Za-z]{2,63}.

This adjustment is applied consistently in the API DTO (GetDomainStatusDto) and in the app’s Zod schemas used by the custom-domain form and DNS check action, reducing false rejections for valid domains like .technology.

Written by Cursor Bugbot for commit 3d742ca. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Feb 4, 2026 5:00pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal Skipped Skipped Feb 4, 2026 5:00pm

Request Review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@chasprowebdev chasprowebdev changed the title [dev] [chasprowebdev] chas/update-domain-validation [HOTFIX] Update the validation logic for trust domain Feb 4, 2026
@Marfuen Marfuen merged commit d88c824 into main Feb 4, 2026
11 checks passed
@Marfuen Marfuen deleted the chas/update-domain-validation branch February 4, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants