Skip to content

🤖 Add DNS Validation and Resolution Logic#363

Merged
doubleangels merged 1 commit intodevfrom
autofix/add-dns-validation-and-resolution-logic
Jan 21, 2025
Merged

🤖 Add DNS Validation and Resolution Logic#363
doubleangels merged 1 commit intodevfrom
autofix/add-dns-validation-and-resolution-logic

Conversation

@seer-by-sentry
Copy link
Copy Markdown
Contributor

👋 Hi there! This PR was automatically generated by Autofix 🤖

This fix was triggered by DoubleAngels

Fixes NEXTDNS-MANAGER-1AT

This pull request introduces a new utility class, DNSResolver, which provides methods to validate NextDNS subdomains and resolve hostnames with retry logic. In VisualIndicator.java, logic has been added to validate the hostname of requests to ensure they are valid NextDNS subdomains. If the hostname is invalid, a message is sent to Sentry. Additionally, the code now attempts to resolve the hostname of requests, capturing errors if resolution fails, and handling specific network exceptions to improve resilience and error reporting.

If you have any questions or feedback for the Sentry team about this fix, please email autofix@sentry.io with the Run ID: 3404.

@doubleangels doubleangels changed the base branch from main to dev January 21, 2025 21:23
@doubleangels doubleangels marked this pull request as ready for review January 21, 2025 21:23
@doubleangels doubleangels merged commit fc84798 into dev Jan 21, 2025
@doubleangels doubleangels deleted the autofix/add-dns-validation-and-resolution-logic branch January 21, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant