Internet Explorer is failing "Object doesn't support property or method 'privBytes'" about elliptic HOT 13 CLOSED

Interesting... What IE version is this? May I ask you to post some code-context from unittests-bundle.js:10979?

from elliptic.

it apply to all the internet explorer versions from 8 to 11 :(

the context is:

  function testFactory(i) {
    it('can compute correct a and A for secret: ' + i, function() {
      var test = derivations[i];
      var secret = utils.toArray(test.secret_hex, 'hex');
      var key = ed25519.keyFromSecret(secret);
      assert.equal(toHex(key.privBytes()), test.a_hex);
      var xRecovered = toHex(ed25519.encodeInt(
                             ed25519.decodePoint(key.pubBytes()).getX()));
      assert.equal(xRecovered, test.A_P.x);
      assert.equal(toHex(key.pubBytes()), test.A_hex);
    });
  }

seems so that the key = ed25519.keyFromSecret(secret); is not returning a correct object containing privBytes method.

from elliptic.

i think i'm near to the point;

cachedProperty(KeyPair, function pubBytes() {
  return this.eddsa.encodePoint(this.pub());
});

i'm thinking that the cachedProperty is not working on IE; possible?

function cachedProperty(obj, computer) {
  var name = computer.name;
  var key = '_' + name;
  obj.prototype[name] = function cachedProperty() {
    return this[key] !== undefined ? this[key] :
           this[key] = computer.call(this);
  };
}

if this was working in past i think that probably the bug has been inserted in this commit: 49533ee

from elliptic.

Probably the .name is empty?

from elliptic.

Seems to be non-standard: http://stackoverflow.com/questions/6903762/function-name-not-supported-in-ie

from elliptic.

bingo!

from elliptic.

Fix is coming in a bit.

from elliptic.

The regression tests will come with your patch, though 😢

from elliptic.

May I ask you to give a try to d3049b8 ?

from elliptic.

Thanks!

from elliptic.

K! going to rebase my patch on current master, this way recommitting i will test it in the cloud!

from elliptic.

verified that tha this bug is solved! thanks to you!

from elliptic.

no problem!

from elliptic.