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

test: adjust common/internet addresses.CNAME_HOST #49109

Closed
wants to merge 1 commit into from

Conversation

LiviaMedeiros
Copy link
Contributor

Right now blog.nodejs.org doesn't have CNAME records.

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Aug 11, 2023
@richardlau
Copy link
Member

Right now blog.nodejs.org doesn't have CNAME records.

It does but I see that someone has turned proxying back on for it which breaks the test (#48868, nodejs/build#3404 (comment)):
image

I've unproxied it again -- I'd already previously added a comment to the entry which was obviously missed.

cc @nodejs/build-infra

@LiviaMedeiros
Copy link
Contributor Author

Oh, missed that issue. Thanks!
Can confirm that (await dns.promises.resolveCname('blog.nodejs.org'))[0] === 'nodejs.org' works, closing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants