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

Can't build on node 0.12 #22

Open
chrisyip opened this issue May 22, 2015 · 6 comments
Open

Can't build on node 0.12 #22

chrisyip opened this issue May 22, 2015 · 6 comments

Comments

@chrisyip
Copy link

node-xmpp <=0.15 doesn't support node 0.12, node-gyp will throw errors.

So I tried to upgrade node-xmpp to 1.0.0-alpha2. Build succeeded, but I could not pass the tests:

> [email protected] test /Users/Chris/Workspace/node/superfeedr-node
> export NODE_ENV=test; mocha

Cannot load StringPrep-0.7.0 bindings (using fallback). You may need to `npm install node-stringprep`
Cannot load StringPrep-0.7.0 bindings (using fallback). You may need to `npm install node-stringprep`

  createCredentials() is deprecated, use tls.createSecureContext instead
/Users/Chris/Workspace/node/superfeedr-node/node_modules/mocha/lib/runner.js:93
  test.failed = true;
              ^
TypeError: Cannot set property 'failed' of undefined
    at Runner.fail (/Users/Chris/Workspace/node/superfeedr-node/node_modules/mocha/lib/runner.js:93:15)
    at process.<anonymous> (/Users/Chris/Workspace/node/superfeedr-node/node_modules/mocha/lib/runner.js:354:10)
    at emitOne (events.js:77:13)
    at process.emit (events.js:169:7)
    at process._fatalException (node.js:211:26)
    at emitOne (events.js:77:13)
    at Connection.emit (events.js:169:7)
    at Connection.onStanza (/Users/Chris/Workspace/node/superfeedr-node/node_modules/node-xmpp/node_modules/node-xmpp-core/lib/connection.js:352:14)
    at StreamParser.<anonymous> (/Users/Chris/Workspace/node/superfeedr-node/node_modules/node-xmpp/node_modules/node-xmpp-core/lib/connection.js:221:14)
    at emitOne (events.js:77:13)
    at StreamParser.emit (events.js:169:7)
    at SaxExpat.<anonymous> (/Users/Chris/Workspace/node/superfeedr-node/node_modules/node-xmpp/node_modules/node-xmpp-core/lib/stream_parser.js:56:22)
    at emitOne (events.js:77:13)
    at SaxExpat.emit (events.js:169:7)
    at null.<anonymous> (/Users/Chris/Workspace/node/superfeedr-node/node_modules/node-xmpp/node_modules/ltx/lib/sax/sax_expat.js:16:14)
    at emitOne (events.js:77:13)
    at emit (events.js:169:7)
    at Parser.parse (/Users/Chris/Workspace/node/superfeedr-node/node_modules/node-xmpp/node_modules/node-expat/lib/node-expat.js:23:22)
    at SaxExpat.write (/Users/Chris/Workspace/node/superfeedr-node/node_modules/node-xmpp/node_modules/ltx/lib/sax/sax_expat.js:27:22)
    at StreamParser.write (/Users/Chris/Workspace/node/superfeedr-node/node_modules/node-xmpp/node_modules/node-xmpp-core/lib/stream_parser.js:125:21)
    at Connection.onData (/Users/Chris/Workspace/node/superfeedr-node/node_modules/node-xmpp/node_modules/node-xmpp-core/lib/connection.js:288:21)
    at emitOne (events.js:82:20)
    at TLSSocket.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:145:16)
    at TLSSocket.Readable.push (_stream_readable.js:109:10)
    at TLSWrap.onread (net.js:509:20)
@julien51
Copy link
Member

I think we'll need to make sure @node-xmpp works with Node 0.12.0. Any idea @lloydwatkin ?

@lloydwatkin
Copy link

@chrisyip you'll need to upgrade to a more recent alpha than alpha2.
Also @julien51 rather than use node-xmpp you should now be using one of the sub-components (as appropriate) node-xmpp-client/component/server.

I think we need to do some upgrades for the latest versions of 0.12 (due to v8 changes) but I've been really busy. @sonnyp or @chrisrock may be able to get these out quicker than me.

@julien51
Copy link
Member

Awesome! Thansk @lloydwatkin for the quick update. I'll do the dependency change as soon as I find a minute. Feel free to submit a PR @chrisyip :)

@chrisyip
Copy link
Author

@lloydwatkin you mean we should wait for node-xmpp aplha3 or higher?

@davepenfold
Copy link

Any ideas why I get a "XMPP authentication failure" for the nodesample account which is being used in the tests?

Here's what I have so far: https://github.com/davepenfold/superfeedr-node

@julien51
Copy link
Member

Dave,
This happens when you use credentials which are not right, like a bad
password or a token which does not allow for XMPP.
Thanks

On Mon, Jun 22, 2015 at 1:42 AM, Dave Penfold [email protected]
wrote:

Any ideas why I get a "XMPP authentication failure" for the nodesample
account which is being used in the tests?

Here's what I have so far: https://github.com/davepenfold/superfeedr-node


Reply to this email directly or view it on GitHub
#22 (comment)
.

Julien Genestoux
Superfeedr https://superfeedr.com/
[email protected]
4158306574

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants