search

centrifugal / jscent

Node.js client to interact with Centrifugo v1 HTTP API
MIT License
11 stars 5 forks source link

fixed deprecated usage of node-uuid in request dep #9

Closed skarbovskiy closed 6 years ago

skarbovskiy commented 6 years ago

2.45 version of request module currently produce 

    console.warn node_modules/node-uuid/uuid.js:48
      [SECURITY] node-uuid: crypto not usable, falling back to insecure Math.random()

more details here https://github.com/request/request/pull/2182

FZambia commented 6 years ago

Hello @skarbovskiy! Thanks a lot!

I hope with Centrifugo v2 developers will be able to integrate with Centrifugo a bit easier. Because there is no request signing. So sending API request is just sending simple JSON object. And tokens can be generated using any JWT library.

skarbovskiy commented 6 years ago

waiting for v2 :)

skarbovskiy commented 6 years ago

@FZambia could u please upgrade published version

FZambia commented 6 years ago

@skarbovskiy sure, just did it.