Syntax error when running with sudo

[gustavooliveiraf]

Support plan

• is this issue currently blocking your project? (yes/no): yes
• is this issue affecting a production system? (yes/no): yes

Context

• node version: 12.14.0
• module version with issue: 17.0.0
• environment (e.g. node, browser, native): node
• used with (e.g. hapi application, another framework, standalone, ...): hapi

What are you trying to achieve or the steps to reproduce?

sudo node app.js

What was the result you got?

.../node_modules/@hapi/joi/lib/errors.js:246
    isJoi = true;
            ^

SyntaxError: Unexpected token =

What result did you expect?

run successfully

[Marsup]

I'm going to guess your node version is not what you expect with sudo because you're using nvm or something alike. If you want to preserve the node version you need to use sudo -E.

[Rapol]

Context
node version: 10.14.1
module version with issue: 17.0.0
environment (e.g. node, browser, native): node
used with (e.g. hapi application, another framework, standalone, ...): standalone

Im running into the same problem with or without sudo. I was able to reproduce it outside our project by just creating a file that just imports joi 17.0.0. I downgraded to 16.1.0 and it works

test/node_modules/@hapi/joi/lib/errors.js:246
    isJoi = true;
          ^

SyntaxError: Unexpected token =
    at new Script (vm.js:79:7)
    at createScript (vm.js:251:10)
    at Object.runInThisContext (vm.js:303:10)
    at Module._compile (internal/modules/cjs/loader.js:656:28)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:699:10)
    at Module.load (internal/modules/cjs/loader.js:598:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:537:12)
    at Function.Module._load (internal/modules/cjs/loader.js:529:3)
    at Module.require (internal/modules/cjs/loader.js:636:17)
    at require (internal/modules/cjs/helpers.js:20:18)

[Marsup]

Joi 17 release notes explicitly specified that it's only compatible with node 12+, so as expected.

[Rapol]

Ah, sorry about that. Tried to find the release notes before commenting but couldn't find them. Would it be possible to add the compatible node versions in the package.json?

[Marsup]

Unlikely, we had too many problems with either npm or yarn when doing that.

[Marsup]

And I forgot, but you do have that information here: https://hapi.dev/family/joi/.