-
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
Unable to clear redirect from force_ssl #40
Comments
Hi @kbbpartners. Maybe something is redirecting ahead of your Heroku app; what's your DNS setup? |
Edited for brevity. The salient points have been left for future reference. I tried turning off force_ssl and using This time, it failed with Note that the challenge set on Heroku in this case was
So now I'm wondering why is |
I'm going to close this issue, although we were unable to resolve this issue. In the end, here is where we left it:
|
I'm unsure why there is a mismatch between your The middleware is aware of the current variables because they've been updated by the Heroku task. Assuming you are updating the environment variables for the correct app (the one responding to web requests), the app will restart when the variables are changed via the API and will have the new values. Is there something about your Heroku configuration that means this restart doesn't happen in the normal manner? |
@kbbpartners I've had an idea that may help fix this, and I'm tracking it with issue #41, I'll try and get a fix out for you to retry. |
@kbbpartners please can you try v1.1.1. |
Thanks! I'll have a look ASAP. |
Hey @kbbpartners, I noticed a bug with my fix. If you haven't already, don't bother trying 1.1.1, and just try v1.1.3 :) |
No problem—we hit a LetsEncrypt rate limit during our efforts to correct this, so we're in a holding pattern for a few days anyway. |
OK—I can confirm that this worked. However, I noticed one minor issue. We have two Custom Domains and then the standard-issue Heroku Domain. We only need certificates for the Custom Domains, so the third certificate (for the Heroku Domain) is superfluous for us—we never use that subdomain. I'm guessing this is probably related to #44 Anyway, thank you! |
Per the README, I've got
config.middleware.insert_before ActionDispatch::SSL, Letsencrypt::Middleware
in config/application.rb. Indeed, runningheroku run rake middleware
shows:But,
heroku run rake letsencrypt:renew
results in:This is on ruby 2.3.1, Rails 4.2.6.
Have I missed something?
The text was updated successfully, but these errors were encountered: