Skip to content
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

fix: move ns to answers section #89

Merged
merged 3 commits into from
Nov 5, 2024
Merged

fix: move ns to answers section #89

merged 3 commits into from
Nov 5, 2024

Conversation

fauno
Copy link
Collaborator

@fauno fauno commented Nov 5, 2024

i was checking against other nameservers and realised the ns records belongs to the answers section, though it's a little weird to always respond with TXT and NS records independently of the actual question. but i think it'll require further fiddling with @types/dns2

@fauno
Copy link
Collaborator Author

fauno commented Nov 5, 2024

@RangerMauve let me know when you deploy, and also could you check that the host config is api.distributed.press? it's answering "localhost" now

@RangerMauve RangerMauve merged commit 7935269 into main Nov 5, 2024
2 checks passed
@RangerMauve
Copy link
Contributor

Deploying

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.

2 participants