-
Notifications
You must be signed in to change notification settings - Fork 33
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
404 on Challenge File Request #5
Comments
Issue was the actually due to line 54: hostname = heroku.domain.list(heroku_app).first['hostname'] By grabbing the I was able to workaround the issue by removing the apex domain from the listing, which isn't ideal: I will need to leave it removed in order for automatic renewal to work. |
Hmm. Surely if your apex is being handled by your DNS provider it doesn't make sense for you to tell Heroku it is responsible for that domain? |
Also are you sure that's the issue? Surely a request to your apex domain, if redirected correctly, wouldn't have resulted in a 404? |
I'm getting the same error. if I run @coreyward are you on Rails 5? I've tried to set up Let's Encrypt manually, I can't seem to get the route working with a leading dot (.well-known). |
@jalada I do believe that this was the issue. I ran the rake task multiple times to no avail, and after removing the bare domain from the Heroku list, ran it again successfully. I suspect the response from the domains endpoint is normalized by alphabetizing, which would result in the bare domain being first in the list. I will typically add the bare domain to Heroku as a matter of completeness. If the client ends up making a change to their DNS records, I don't want to see non-www-prefixed requests simply fail. It also prevents confusion in the potential case of someone else adding the domain to another application in Heroku. If this gem used the first in the list from @heyogrady Yes, I'm on Rails 5. |
Thank you for the further investigation @coreyward. You’re right, it would probably be better to use the I’ll happily accept a pull request changing it. If not I will get round to it as soon as I can |
Line 55 is the
open
call:The text was updated successfully, but these errors were encountered: