How to avoid "Invalid user error"

[haosdent]

Hello, I try to write a third part plugin like packages/rocketchat-hubot. But I meet this error when I call Meteor.call "sendMessage".

Exception in callback of async function: Error: [methods] sendMessage -> Invalid user [invalid-user]

I check the code of rocketchat-hubot, but seem it also don't set userid. I am confuse here, how could I avoid this error. Thank you in advance.

[haosdent]

My code snippet is

        Meteor.call "sendMessage",
            u:
                username: name
            to: target
            msg: content
            rid: 'EtomNKRwxp6mELsGGLHa4ybHScj2WZoo3K'

I have already wrap this function by Meteor.bindEnvironment before.

[sampaiodiego]

maybe @Sing-Li can answer this?

[engelgabriel]

@haosdent did you create a user for Hubot? You need to register yourself as Hubot, and set the login credentials on the hubot server via environment variables.

[engelgabriel]

We need to add a default Hubot user and generate a random token, visible to the admin

[haosdent]

Thank you for your reply. @Sing-Li @engelgabriel .
In fact, I try to write a irc plugin for rocket.chat. https://github.com/haosdent/rocketchat-irc/blob/master/irc.server.coffee

And it could works although it only have the basic communicate function.

Channel message:

Direct message:

[haosdent]

I add a receiveMessage to Meteor.methods in server directory. But I am not sure whether it is the best way or not.

[engelgabriel]

That looks cool! Lets me try it and I'll give you some feedback. About the receiveMessage, I had a similar idea... let's talk about it after I have had read the code.

[haosdent]

Thank you for your great help.

[engelgabriel]

@haosdent do you want make a pull request with yous script? I'd be glad to accept it. We just need to turn all the configuration loaded form the DB... so we can setup using an admin interface.

[haosdent]

Sure and very gladful you want to accept it. Let me fix some minor issue of my scripts and create a merge request.

[Sing-Li]

Cool! Now we can 'bridge in' mainstream OSS project IRC channels 👍

[engelgabriel]

@haosdent

I've split the /server/methods/sendMessage.coffee method into the server method that can be called by the client (that has to pass all auth/permissions check) and the rest main functionality is now on a function in /packages/rocketchat-lib/server/sendMessage.coffee that can only be called by server side scripts (without auth/permissions checks)

This should eliminate the need of your /server/methods/receiveMessage.coffee

Can you please review it? And send a PR migrating to the new method?

[haosdent]

Thank you very much! I see it, let me migrate.