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

effector.js.org #2854

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@goodmind
Copy link
Contributor

goodmind commented Mar 5, 2019

So, should I send TXT dns record in personal messages for this to work?

@MattIPv4
Copy link
Collaborator

MattIPv4 left a comment

Commit valid and site has content based on https://effector.now.sh

I trust your now.sh config is setup to work with your requested js.org subdomain.

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 5, 2019

@goodmind This subdomain was requested by @zerobias (and it looks like it is still active https://github.com/zerobias/effector/blob/gh-pages/CNAME)
@zerobias has has to make or approve this PR before we can start the process.

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 5, 2019

So, should I send TXT dns record in personal messages for this to work?

I would guess it not a big security problem to just post them here.

@MattIPv4

This comment has been minimized.

Copy link
Collaborator

MattIPv4 commented Mar 5, 2019

@indus The previous PR for this subdomain was created by @goodmind not zerobias
#2801

@indus indus removed the awaiting response label Mar 5, 2019

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 5, 2019

Sorry, my mistake. I've made the change. But looks like we need the TXT record to make it work. @goodmind If you don't want to just put the record here you can also send it to admin@js.org if you like (but you should give me a hint here to make me search for it in the flood of spam).

@zerobias

This comment has been minimized.

Copy link

zerobias commented Mar 6, 2019

Hello, I'm confirming we are working on the same project

Name Type Value
_now TXT QmNhSa4YaCWADTnU27WK4LncFKM3nM5uUZuoNph9UY7xCA
@goodmind

This comment has been minimized.

Copy link
Contributor Author

goodmind commented Mar 6, 2019

@indus can you merge?

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 6, 2019

I've added the TXT record: https://mxtoolbox.com/SuperTool.aspx?action=txt%3a_now.js.org&run=toolpage Could you confirm it is working for you?

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 10, 2019

@goodmind Without a response from your side I will have to close this.

@goodmind

This comment has been minimized.

Copy link
Contributor Author

goodmind commented Mar 10, 2019

@indus I'll check

@goodmind

This comment has been minimized.

Copy link
Contributor Author

goodmind commented Mar 10, 2019

Error! The domain effector.js.org could not be verified due to the following reasons: [25s]

  a) Nameservers verification failed since we see a different set than the intended set:
     Intended Nameservers    Current Nameservers
     b.zeit-world.co.uk      -                      ✘
     d.zeit-world.org        -                      ✘
     e.zeit-world.com        -                      ✘
     f.zeit-world.net        -                      ✘

  b) DNS TXT verification failed since found no matching records.
     name        type        value
     _now        TXT         QmNhSa4YaCWADTnU27WK4LncFKM3nM5uUZuoNph9UY7xCA
@goodmind

This comment has been minimized.

Copy link
Contributor Author

goodmind commented Mar 10, 2019

@indus maybe CNAME should point to alias.zeit.co instead of effector.now.sh

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 10, 2019

@goodmind I tried "alias.zeit.co" but it doesn't look like it is working. Do you have to trigger some reevaluation at you host?
Nevertheless this sounds like the correct option: https://zeit.co/docs/v2/domains-and-aliases/adding-a-domain/#preserving-existing-nameservers-with-cname

@goodmind

This comment has been minimized.

Copy link
Contributor Author

goodmind commented Mar 10, 2019

@indus is it alias.zeit.co now? I tried verifying domain again but same error

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 10, 2019

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 11, 2019

@goodmind are you still trying to get this configured correctly. Besides changing values in JS.ORGs zonefile I'm unable to help you with this. I will have to close this if there is no solution to the problems with your hosting solution.

@goodmind

This comment has been minimized.

Copy link
Contributor Author

goodmind commented Mar 11, 2019

Yeah, contacting now support 😒

@MattIPv4

This comment has been minimized.

Copy link
Collaborator

MattIPv4 commented Mar 14, 2019

@goodmind Any update?

@goodmind

This comment has been minimized.

Copy link
Contributor Author

goodmind commented Mar 14, 2019

No, sorry, feel free to close it

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 14, 2019

@goodmind Does that mean you want to remove the subdomain.

@goodmind

This comment has been minimized.

Copy link
Contributor Author

goodmind commented Mar 14, 2019

No, leave it as github

@MattIPv4

This comment has been minimized.

Copy link
Collaborator

MattIPv4 commented Mar 14, 2019

@goodmind Appreciate it, apologies for the confusion here.

@indus Is the DNS back to pointing at the GitHub target so this can be closed?

@indus

This comment has been minimized.

Copy link
Member

indus commented Mar 14, 2019

@MattIPv4 Yes - everything is the same as where this started.

@indus indus closed this Mar 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.