add note about why dns resolution order is necessary

This commit is contained in:
H. Shay 2023-11-14 12:42:18 -08:00
parent 6463118ebc
commit c5f0366a5f
3 changed files with 3 additions and 0 deletions

View File

@ -10,6 +10,7 @@ interface Context extends Mocha.Context {
appservice?: MjolnirAppService
}
// Necessary for CI: Node 17+ defaults to using ipv6 first, but Github Actions does not support ipv6
dns.setDefaultResultOrder('ipv4first');
describe("Test that the app service can provision a mjolnir on invite of the appservice bot", function () {

View File

@ -13,6 +13,7 @@ interface Context extends Mocha.Context {
moderator?: MatrixClient
}
// Necessary for CI: Node 17+ defaults to using ipv6 first, but Github Actions does not support ipv6
dns.setDefaultResultOrder('ipv4first');
describe("Test that the app service can provision a mjolnir when requested from the web API", function () {

View File

@ -3,6 +3,7 @@ import { makeMjolnir, teardownManagementRoom } from "./mjolnirSetupUtils";
import { register } from "prom-client";
import dns from 'node:dns';
// Necessary for CI: Node 17+ defaults to using ipv6 first, but Github Actions does not support ipv6
dns.setDefaultResultOrder('ipv4first');
// When Mjolnir starts (src/index.ts) it clobbers the config by resolving the management room