Fix for #90 (and hopefully #91)

[idiidk]

I can't reproduce #91 so I can't really test if this fixes it. It does fix #90 tho

[gruijter]

I hope this works. But I cannot test this, since I don't understand the babel stuff. How do I make a version to test this with?

[lieuwex]

Thanks!

[lieuwex]

@gruijter npm install && npm run build in the Magisterjs folder should build the needed filed in lib/.

[gruijter]

thanx. I did build and got a lib/. But when testing I get

TypeError: setToken(...).then is not a function
    at \node_modules\magister.js\lib\magister.js:522:36

[netlob]

I'm getting the same error as @gruijter:

something went wrong: TypeError: setToken(...).then is not a function
    at \node_modules\magister.js\lib\magister.js:522:36
    at Generator.next (<anonymous>)
    at step (\node_modules\magister.js\lib\magister.js:132:191)
    at \node_modules\magister.js\lib\magister.js:132:437
    at new Promise (<anonymous>)
    at \node_modules\magister.js\lib\magister.js:132:99
    at Magister.login (\node_modules\magister.js\lib\magister.js:589:5)
    at Promise.resolve.then (\node_modules\magister.js\lib\magister.js:624:28)
    at process._tickCallback (internal/process/next_tick.js:68:7)

[lieuwex]

I've fixed the issues, and published a new version to npm. The codepath using tokens wasn't tested, I should review the code better before I quickly merge it when I wake up lol. Sadly I don't have a Magister account so I can only read the code and guess if it's correct or not.

[gruijter]

@lieuwex Thx for your help! But unfortunately another error appears now:

TypeError: Cannot use 'in' operator to search for 'SecondsLeft' in SecurityToken Expired
    at promise.then.then.res (\node_modules\magister.js\lib\http.js:151:22)

[lieuwex]

@gruijter huh funny, didn't expect that to happen. I've opened #94 because I have to first fix another issue before I can fix this.

[idiidk]

Oh shit, completere my fault to open a pull before testing fully. Sorry @lieuwex

[gruijter]

When using username/password #91 is not solved. Using the token results in #94.

[gruijter]

I can confirm that with alpha4 #91 is still an issue.