-
-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(console): DNS validation for custom domain CNAME records. #2357
Conversation
packages/utils/errors.ts
Outdated
@@ -87,6 +92,7 @@ const getErrorFromTRPCClientError = (error: any): Error => { | |||
} | |||
} else if (error.data && 'zodError' in error.data) { | |||
const cause = (error.data as { zodError: object }).zodError | |||
console.debug('ZOD', cause) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intended?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed
for (const dnsRec of stored.dns_records) { | ||
dnsRec.value = await getDNSRecordValue(dnsRec.name, dnsRec.record_type) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would this block need exception handling?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as other comment.
for (const dnsRec of stored.dns_records) { | ||
dnsRec.value = await getDNSRecordValue(dnsRec.name, dnsRec.record_type) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would this block need exception handling?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As per convo, it would get picked up by the next alarm run until it eventually validates fully. Any errors will propagate up to the caller in Console, which will log for visibility.
* DNS validation for custom domain CNAME records. * Custom domain validation includes CNAME validation * Removed unnecessary debug statement.
Description
Adds indicator for validating CNAME records on the custom domain page. CNAME validitiy check is now included as part of the custom domain validity indicator on the page. The same is done in Passport to check whether custom domain is active.
Related Issues
Testing
Checklist