Update bn.js to address Buffer deprecation

[exoego]

Closes #196

[exoego]

Hmm, some tests failed due to this changes in bn.js.
How should these tests be addressed?
Does this indicated an issue in elliptic, or in bn.js ?

EDIT:
I fixed the tests in elliptic 83ff218.

[fanatid]

package-lock.json should not be added. There still no decision how this should be handled. See #191

[exoego]

@fanatid
It seems that Travis complains unless package-lock.json included

[fanatid]

Oh, sorry. package-lock.json in this repo now, that's why Travis show error. On each dependency update package-lock.json should be updated too...

[exoego]

Ok, I will add package-lock.json back.

[exoego]

This indicates asn1.js 4.10.1 still uses bn.js 4.11.8, which cause another Buffer deprecation.
So I opened a PR indutny/asn1.js#116 to update bn.js in asn1.js too.

[exoego]

PR is opened.
browserify/browserify-rsa#12

[exoego]

browserify/browserify-sign#45

[exoego]

browserify/createECDH#13

[exoego]

browserify/diffie-hellman#31

[exoego]

indutny/miller-rabin#11

[exoego]

browserify/publicEncrypt#20

[coveralls]

Coverage decreased (-0.006%) to 89.114% when pulling 1894eb9 on exoego:buffer-deprecation into 71e4e8e on indutny:master.

[exoego]

Hello core developers, how should I proceed this and related PRs?

[exoego]

@fanatid
Updated bn.js to 5.1.1 since newer version have been released while waiting review.

[exoego]

Closing as a duplicate, since a maintainer opened a same PR in #210