Skip to content

Conversation

roman-koshchei
Copy link
Contributor

I should say, that I am not the author of the library.
BUT the author is interested in js.org, so I just help him.
You can check it: github.com/piranha/twinspark-js/issues/23.

@MattIPv4 MattIPv4 changed the title Add TwinSpark twinspark.js.org Apr 18, 2023
Copy link
Member

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi there!

❌ Before we can accept your domain request, you need to create a file named CNAME in the root of your github pages site (this may not be the root of your repository). 📝
This file should contain a single line with the full js.org domain you are requesting, in this instance twinspark.js.org.

@MattIPv4 MattIPv4 added the awaiting custom domain This PR is waiting for the GitHub Pages target to set the custom domain label Apr 18, 2023
@piranha
Copy link

piranha commented Apr 18, 2023

@MattIPv4 done :) sorry for the inconvenience!

@indus
Copy link
Member

indus commented Apr 18, 2023

I've already added your requested subdomain to JS.ORGs zonefile. But as you are using a workflow to deploy the page there may be other steps involved to get the CNAME working. FYI: #7853 (comment)

@piranha
Copy link

piranha commented Apr 18, 2023

Riight I've added that to settings, but there is some problem with HTTPS redirect - it redirects to itself.
image

@piranha
Copy link

piranha commented Apr 18, 2023

For anybody that comes across this. I think you need to do the default GitHub pages deployment with the custom domain set. That gets it setup. You can then flip it back to the GitHub actions deployment.

Ohh I'll try that, re-deploying to check if it helps right now.

@piranha
Copy link

piranha commented Apr 18, 2023

Okay I did what that comment says, but I had to be more patient — it started working with Actions workflow while I was waiting for an old-style deploy. :-) I'll revert that stuff back, heh.

@piranha
Copy link

piranha commented Apr 18, 2023

Argh :( moving back to actions deploy destroyed it. Wtf :(

@piranha
Copy link

piranha commented Apr 18, 2023

twinspark.js.org has address 172.67.73.64
twinspark.js.org has address 104.26.8.84
twinspark.js.org has address 104.26.9.84

I guess those are your IPs, not Github's, right? I'm trying to understand where the issue can be...

@MattIPv4
Copy link
Member

Yes, those are Cloudflare IPs, as the domain is proxied through Cloudflare.

@piranha
Copy link

piranha commented Apr 18, 2023

I did the whole dance again and it seems to work. But last two commits did not trigger a workflow run, and I have no idea why. 🤣 So I'm wary of declaring this issue a success — it works now, but who knows what will happen when (if?) workflow runs.

@indus indus added add This PR/issue is regarding adding a new subdomain organisation This PR/issue is regarding a target on a GitHub organisation project This PR has a target of a project on an organisation or user account and removed awaiting custom domain This PR is waiting for the GitHub Pages target to set the custom domain labels Apr 18, 2023
@indus
Copy link
Member

indus commented Apr 18, 2023

@piranha the popularity of workflows has made this whole procedure a bit more challenging for us as well. It is not allways clear what causes problems. But after getting it work initially we never heard any complaints from subdomain requesters 🤞
Welcome https://twinspark.js.org

@indus indus merged commit 8457711 into js-org:master Apr 18, 2023
@piranha
Copy link

piranha commented Apr 18, 2023

It seems like Github had a hiccup, and now has triggered workflows for all commits and all is well, so 🎉 :-)) Thanks!

It's weird that workflows don't work as well with custom domains, since it seems like old-style way just triggers some built-in workflow...

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

Labels

add This PR/issue is regarding adding a new subdomain organisation This PR/issue is regarding a target on a GitHub organisation project This PR has a target of a project on an organisation or user account

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants