search

multinet-app / multinet-deploy

The ansible deployment tasks for multinet.app
0 stars 0 forks source link

Fix cerbot command #26

Closed JackWilb closed 4 years ago

JackWilb commented 4 years ago

Adds the --standalone flag to the certbot command and handles the nginx server as required.

JackWilb commented 4 years ago

@waxlamp Once I can get this to pass I'll merge

JackWilb commented 4 years ago

This only passed because the cert was already installed. However, I don't feel like bricking the https on the prod server to test this. I think we should just call it good and monitor it for now. What do you think @waxlamp

waxlamp commented 4 years ago

This only passed because the cert was already installed. However, I don't feel like bricking the https on the prod server to test this. I think we should just call it good and monitor it for now. What do you think @waxlamp

I agree. The situation we are watching for mainly occurs the first time a new cert is being set up, is that right?

JackWilb commented 4 years ago

Correct. I'll merge it as it is for now then