diff --git a/lib/internal/process.js b/lib/internal/process.js index 881d91b11e9a73..c435c2e8712a03 100644 --- a/lib/internal/process.js +++ b/lib/internal/process.js @@ -117,6 +117,21 @@ function setupConfig(_source) { if (value === 'false') return false; return value; }); + const processConfig = process.binding('config'); + // Intl.v8BreakIterator() would crash w/ fatal error, so throw instead. + if (processConfig.hasIntl && + processConfig.hasSmallICU && + Intl.hasOwnProperty('v8BreakIterator') && + !process.icu_data_dir) { + const des = Object.getOwnPropertyDescriptor(Intl, 'v8BreakIterator'); + des.value = function v8BreakIterator() { + throw new Error('v8BreakIterator: full ICU data not installed. ' + + 'See https://github.com/nodejs/node/wiki/Intl'); + }; + Object.defineProperty(Intl, 'v8BreakIterator', des); + } + // Don’t let icu_data_dir leak through. + delete process.icu_data_dir; } diff --git a/src/node.cc b/src/node.cc index 6f155a64a387c6..eb8805b2195ad6 100644 --- a/src/node.cc +++ b/src/node.cc @@ -2976,6 +2976,13 @@ void SetupProcessObject(Environment* env, READONLY_PROPERTY(versions, "icu", OneByteString(env->isolate(), U_ICU_VERSION)); + + if (icu_data_dir != nullptr) { + // Did the user attempt (via env var or parameter) to set an ICU path? + READONLY_PROPERTY(process, + "icu_data_dir", + OneByteString(env->isolate(), icu_data_dir)); + } #endif const char node_modules_version[] = NODE_STRINGIFY(NODE_MODULE_VERSION); diff --git a/test/parallel/test-intl-v8BreakIterator.js b/test/parallel/test-intl-v8BreakIterator.js new file mode 100644 index 00000000000000..6e25c32cd743d3 --- /dev/null +++ b/test/parallel/test-intl-v8BreakIterator.js @@ -0,0 +1,15 @@ +'use strict'; +require('../common'); +const assert = require('assert'); + +if (global.Intl === undefined || Intl.v8BreakIterator === undefined) { + return console.log('1..0 # Skipped: no Intl'); +} + +try { + new Intl.v8BreakIterator(); + // May succeed if data is available - OK +} catch (e) { + // May throw this error if ICU data is not available - OK + assert.throws(() => new Intl.v8BreakIterator(), /ICU data/); +}