Skip to content

Conversation

sujaya-sys
Copy link
Contributor

@sujaya-sys sujaya-sys commented Oct 6, 2025

Given that DNS monitors are still in development, these docs require a final review and update once the feature has been finalized. A few notes on current gaps:

  • The section "Troubleshooting Common Issues" is currently AI generated. Would be great to get someone with DNS expertise to provide feedback on whether these are helpful examples.

  • Any copy related to JSON responses and JSON path assertions was a bit of a guessing game on my end, the snippets and assertion examples would need to be reviewed.

  • Anything else worth mentioning in the dev docs? e.g. any caching or lookup behavior that might be worth calling out for more technical folks?

@sbezludny sbezludny marked this pull request as ready for review October 14, 2025 15:22
Copy link
Contributor

Choose a reason for hiding this comment

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

This UI has changed a tiny bit (inputs fit 100% width of section )

API Spec Updater and others added 2 commits October 15, 2025 09:32
Copy link

@ejanusevicius ejanusevicius left a comment

Choose a reason for hiding this comment

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

Looks good!

Current limitations:

- **Private locations**: DNS monitors do not currently support private locations. You can only run DNS monitors from Checkly's global public locations.
- **Protocol**: DNS queries use UDP by default with automatic TCP fallback. You can pin to UDP-only or TCP-only if needed.

Choose a reason for hiding this comment

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

nit: I don't think that is the case, we explicitly set a protocol which is used for the DNS query. 🤔

@sbezludny sbezludny merged commit f20ff78 into main Oct 15, 2025
3 checks passed
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.

4 participants