conduit: Fix acme issue #91

Manually merged
tlater merged 1 commits from tlater/fix-conduit-acme into master 2023-10-13 05:12:02 +01:00

letsencrypt will prod on port 80 to verify the domain. listen
overrides addSSL, so none of the NixOS modules' setup will actually
work.

This means the conduit virtualhost never listened on port 80, and
couldn't verify letsencrypt requests.

How this ever worked is beyond me, but this commit resolves the
problems (don't worry, forceSSL does what it says on the tin and
overrides the listen again).

letsencrypt will prod on port 80 to verify the domain. `listen` overrides `addSSL`, so none of the NixOS modules' setup will actually work. This means the conduit virtualhost never listened on port 80, and couldn't verify letsencrypt requests. How this *ever* worked is beyond me, but this commit resolves the problems (don't worry, `forceSSL` does what it says on the tin and overrides the `listen` again).
tlater added 1 commit 2023-10-13 05:11:56 +01:00
759a9c7c0c
conduit: Fix acme issue
letsencrypt will prod on port 80 to verify the domain. `listen`
overrides `addSSL`, so none of the NixOS modules' setup will actually
work.

This means the conduit virtualhost never listened on port 80, and
couldn't verify letsencrypt requests.

How this *ever* worked is beyond me, but this commit resolves the
problems (don't worry, `forceSSL` does what it says on the tin and
overrides the `listen` again).
tlater manually merged commit 759a9c7c0c into master 2023-10-13 05:12:02 +01:00
tlater deleted branch tlater/fix-conduit-acme 2023-10-13 05:12:11 +01:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: tlaternet/tlaternet-server#91
There is no content yet.