Polyfill os.hostname() on Windows 7

Co-authored-by: Fedor Indutny <79877362+indutny-signal@users.noreply.github.com>
This commit is contained in:
automated-signal 2022-06-20 11:15:43 -07:00 committed by GitHub
parent 4e92ea07d9
commit a27dee79f7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 1 deletions

View File

@ -396,6 +396,16 @@ async function prepareUrl(
);
}
let hostname: string;
try {
// os.hostname() doesn't work on Windows 7 anymore
// See: https://github.com/electron/electron/issues/34404
hostname = os.hostname();
} catch {
hostname = 'Desktop';
}
const urlParams: RendererConfigType = {
name: packageJson.productName,
locale: getLocale().name,
@ -411,7 +421,7 @@ async function prepareUrl(
environment: enableCI ? Environment.Production : getEnvironment(),
enableCI,
nodeVersion: process.versions.node,
hostname: os.hostname(),
hostname,
appInstance: process.env.NODE_APP_INSTANCE || undefined,
proxyUrl: process.env.HTTPS_PROXY || process.env.https_proxy || undefined,
contentProxyUrl: config.get<string>('contentProxyUrl'),