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

[WiP] Support node v6.2.0 #90

Closed
wants to merge 1 commit into from
Closed

Conversation

michielbdejong
Copy link
Contributor

No description provided.

@michielbdejong
Copy link
Contributor Author

Switching node from v6.9.1 to v6.2.0 makes several integration tests fail, as follows:

[michiel@mbp ~/gh/interledgerjs/ilp (mj-support-node-v6.2.0)]$ docker run -it michielbdejong/five-bells-integration-test /bin/bash
server starting
 * Starting Apache httpd web server apache2                                                                                                                                          AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using 172.17.0.2. Set the 'ServerName' directive globally to suppress this message
 *
root@81f18fe9aeb7:/app# nvm install 6.2.0
Downloading and installing node v6.2.0...
Downloading https://nodejs.org/dist/v6.2.0/node-v6.2.0-linux-x64.tar.xz...
######################################################################## 100.0%
Computing checksum with sha256sum
Checksums matched!
Now using node v6.2.0 (npm v3.8.9)
root@81f18fe9aeb7:/app# src/bin/integration test index


  Basic
ledger:3001 > five-bells-ledger@20.1.1 start /app/integration-test/node_modules/five-bells-ledger
ledger:3001 > node src/app.js

[...]


  1 passing (11s)
  11 failing

  1) Basic send universal payment transfers the funds (by destination amount):
     Error: Invalid IV length
      at Error (native)
      at new Cipheriv (crypto.js:184:16)
      at Object.Cipheriv (crypto.js:182:12)
      at Object.aesEncryptBuffer (node_modules/ilp/src/utils/crypto.js:61:12)
      at createDetails (node_modules/ilp/src/utils/details.js:54:20)
      at Object.createPacketAndCondition (node_modules/ilp/src/lib/transport.js:29:19)
      at Object.createPacketAndCondition (node_modules/ilp/src/lib/ipr.js:36:20)
[...]

@michielbdejong
Copy link
Contributor Author

Continued in #93

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

Successfully merging this pull request may close these issues.

None yet

1 participant