Skip to content

Commit

Permalink
test/nginx: remove checks for well-known/acme-challenge (#748)
Browse files Browse the repository at this point in the history
* this path isn't relevant to these tests, as they do not use letsencrypt SSL certs
* the tests don't make sense, as they're checking for 404s
* in the test scenario, specific handling for this path is removed from redirector.conf in files/nginx/setup-odk.sh
  • Loading branch information
alxndrsn authored Oct 11, 2024
1 parent 79792be commit 29c42ff
Showing 1 changed file with 0 additions and 17 deletions.
17 changes: 0 additions & 17 deletions test/test-nginx.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,23 +6,6 @@ describe('nginx config', () => {
resetBackendMock(),
]));

it('well-known should serve from HTTP', async () => {
// when
const res = await fetchHttp('/.well-known/acme-challenge');

// then
assert.equal(res.status, 301);
assert.equal(res.headers.get('location'), 'https://localhost:9000/.well-known/acme-challenge');
});

it('well-known should serve from HTTPS', async () => {
// when
const res = await fetchHttps('/.well-known/acme-challenge');

// then
assert.equal(res.status, 404);
});

it('HTTP should forward to HTTPS', async () => {
// when
const res = await fetchHttp('/');
Expand Down

0 comments on commit 29c42ff

Please sign in to comment.