Skip to content

refactor: improve dynamic RDAP DNS data#6971

Merged
louislam merged 4 commits intomasterfrom
dev-rdap
Feb 19, 2026
Merged

refactor: improve dynamic RDAP DNS data#6971
louislam merged 4 commits intomasterfrom
dev-rdap

Conversation

@louislam
Copy link
Copy Markdown
Owner

@louislam louislam commented Feb 18, 2026

Summary

The current approach is depending on GitHub workflow, which requires users to keep Uptime Kuma up-to-date. It may be a big problem when we will bump the version to 3, because not everyone are able to upgrade to 3 due to breaking changes.

Manual test seems fine, will add auto test if possible later.

Please follow this checklist to avoid unnecessary back and forth (click to expand)
  • ⚠️ If there are Breaking change (a fix or feature that alters existing functionality in a way that could cause issues) I have called them out
  • 🧠 I have disclosed any use of LLMs/AI in this contribution and reviewed all generated content.
    I understand that I am responsible for and able to explain every line of code I submit.
  • 🔍 Any UI changes adhere to visual style of this project.
  • 🛠️ I have self-reviewed and self-tested my code to ensure it works as expected.
  • 📝 I have commented my code, especially in hard-to-understand areas (e.g., using JSDoc for methods).
  • 🤖 I added or updated automated tests where appropriate.
  • 📄 Documentation updates are included (if applicable).
  • 🧰 Dependency updates are listed and explained.
  • ⚠️ CI passes and is green.

@louislam louislam marked this pull request as ready for review February 19, 2026 04:46
Copilot AI review requested due to automatic review settings February 19, 2026 04:46
@github-actions github-actions bot added the pr:needs review this PR needs a review by maintainers or other community members label Feb 19, 2026

This comment was marked as resolved.

@louislam louislam merged commit 3a46279 into master Feb 19, 2026
24 checks passed
@louislam louislam deleted the dev-rdap branch February 19, 2026 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:needs review this PR needs a review by maintainers or other community members

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants