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

[NS Records] thehcj for cloudflare tunnels #17346

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TheHCJ
Copy link
Contributor

@TheHCJ TheHCJ commented Nov 24, 2024

I need nameserver records as I need to use cloudflare tunnelling, and not having nameserver records could expose my home ip address

Requirements

Unless explicitly specified otherwise by a maintainer or in the requirement description, your domain must pass ALL the indicated requirements above.

Please note that we reserve the rights not to accept any domain at our own discretion.

  • The file is in the domains folder and is in the JSON format.
  • The file's name is all lowercased and alphanumeric.
  • You have completed your website.
  • The website is reachable.
  • The CNAME record doesn't contain https:// or /.
  • There is sufficient information at the owner field.

Website Link/Preview

https://thehcj.is-a.dev

Why I need NS records?

I need nameserver records as I need to use cloudflare tunnelling, and not having nameserver records could expose my home ip address

I need nameserver records as I need to use cloudflare tunnelling, and not having nameserver records could expose my home ip address
@TheHCJ TheHCJ requested a review from a team as a code owner November 24, 2024 20:55
@TheHCJ
Copy link
Contributor Author

TheHCJ commented Nov 24, 2024

I have fixed the issue with the tests validation :)

@DEV-DIBSTER DEV-DIBSTER changed the title add ns records to thehcj for cloudflare tunnels [NS Records] thehcj for cloudflare tunnels Nov 24, 2024
@DEV-DIBSTER
Copy link
Member

@TheHCJ Make sure that domains/_atproto.thehcj.json is deleted before, which you should do in this PR. Since all previous domains have to be deleted before it can be merged.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants