Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UUID has minimum version #40

Closed
brainthinks opened this issue Apr 17, 2017 · 3 comments
Closed

UUID has minimum version #40

brainthinks opened this issue Apr 17, 2017 · 3 comments

Comments

@brainthinks
Copy link

brainthinks commented Apr 17, 2017

My project has UUID version 2.0.1 installed, and because this project's package.json requires any uuid version, it won't install its own version of UUID. I get the following error:

uncaughtException: { Error: Cannot find module 'uuid/v4'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (./node_modules/highcharts-export-server/lib/server.js:38:14)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3) code: 'MODULE_NOT_FOUND' }

After I upgrade my project's version of UUID, I was able to successfully run the highcharts export server.

I'm not sure which version of UUID added the uuid/v4 option, so I'm not sure what the absolute minimum is. UUID version 3.0.1 (the current latest) worked for me though.

Additionally, to prevent other potential breaking changes from other libraries, you may also want to specify minimum version of the other dependencies that are listed as "*".

@cvasseng
Copy link
Contributor

Which version of the export server are you using? UUID has been included as "uuid": ">=3.0.1" in package.json since version 1.0.11 on NPM, and since February 10th in master.

@wergeld
Copy link

wergeld commented Apr 24, 2017

This is found in highcharts-export-server\lib\server.js:

const uuid = require('uuid/v4');

@larsac07
Copy link
Contributor

@wergeld const uuid = require('uuid/v4'); does not mean requiring a version of the uuid package, which was the issue here, but rather the UUID version.

Since updating highcharts-export-server to version 1.0.11 (or commit 54dba5f in master) fixes this problem, I'm considering this issue as closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants