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

Internal Hubot uses obsolete script repository #6640

Closed
wohali opened this issue Apr 10, 2017 · 15 comments
Closed

Internal Hubot uses obsolete script repository #6640

wohali opened this issue Apr 10, 2017 · 15 comments

Comments

@wohali
Copy link

wohali commented Apr 10, 2017

Rocket.Chat Version: 0.54.2
Running Instances: 1
DB Replicaset OpLog: Disabled
Node Version: v4.8.1

The Internal Hubot admin settings page says:

Please enter a comma separated list of scripts to load from https://github.com/github/hubot-scripts/tree/master/src/scripts

However, the top-level README for that repository reads:

DEPRECATED, see github/hubot-scripts#1113 for details

which in turn says:

IMPORTANT: No longer accepting new scripts to this package, create external packages instead

Reading through the history, the hubot-scripts repo was already deprecated in 2015!

On top of this, many of the scripts don't even work anymore, not having been maintained for 3 years or more - see #5952 for details.

Please revisit how scripts can be added to the internal Hubot without using this deprecated (and mostly non-functional) repository.

@JSzaszvari
Copy link
Contributor

JSzaszvari commented Apr 10, 2017 via email

@geekgonecrazy
Copy link
Contributor

geekgonecrazy commented Apr 10, 2017

Yes currently it uses a preseeded folder of scripts compiled in.

We are working on resolving this. The issue here is we can't just let you specify any random script from npm. Because we won't be able.to handle the installation of the scripts.

It'll have to only point to scripts you've dropped into a folder.

We have an issue open for setting an external folder where you can add your own scripts

@wohali
Copy link
Author

wohali commented Apr 10, 2017 via email

@Rohlik
Copy link
Contributor

Rohlik commented Apr 11, 2017

Default scripts can be found here: rocketchat/programs/server/npm/node_modules/meteor/rocketchat_internal-hubot/node_modules/hubot-scripts/src/scripts/
So, if you want add new one, just put it there and activate in web http://your_chat/admin/InternalHubot.
But every RC upgrade will remove your custom scripts!

@wohali
Copy link
Author

wohali commented Apr 11, 2017

Thanks - unfortunately I deploy using Docker so I won't be able to leverage this approach easily. And it still doesn't show me what the default list of scripts is from anywhere in this repo, which is what I was looking for.

@JSzaszvari
Copy link
Contributor

JSzaszvari commented Apr 11, 2017

Mount a local folder on the docker host/storage into

/app/bundle/programs/server/npm/node_modules/meteor/rocketchat_internal-hubot/node_modules/hubot-scripts/src/scripts/

Then you can just pop your choice of scripts in there the scripts in there and they wont go anywhere on upgrades.

@geekgonecrazy geekgonecrazy added this to the Mid-term milestone Apr 26, 2017
@wemu
Copy link

wemu commented May 5, 2017

Hello

a lot of the packaged hubot scripts would already be enough for a lot of things.

are there plans to update at least those? I've tried a few and a lot of them don't do anything or require environment settings (bing api key for a deprecated api) that are unclear to me where to put them.

there are a lot of scripts. my main focus was search, images, youtube, maps (google or bing). what people use the most :)

@JSzaszvari
Copy link
Contributor

@wemu the environment variables are set in the same place you set MONGO_URL, ROOT_URL and the PORT url at the time of deployment / startup

@wemu
Copy link

wemu commented May 6, 2017

ah I see. Thanks! I'll try some more of the scripts - quite fun 👍
yet working versions for google and google images would be much cooler :D

@ghost
Copy link

ghost commented May 10, 2017

I have question, too.
How can I make internal hubot working?
I have clean install of rocket chat, I have enabled Internal Hubot in settings and installed with npm install -g hubot-rocketchat
rocket.cat user seems to be online, and I have that in my log files:
[Wed May 10 2017 10:59:16 GMT+0000 (UTC)] WARNING A script has tried registering a HTTP route while the HTTP server is disabled with --disabled-httpd. Loaded hubot-help/src/help.coffee Loaded hello.coffee Loaded zen.coffee Loaded megusta.coffee
Looks like scripts are being loaded.
But if I am writing, for example, "hello" to rocket.cat user, it doesn`t respond.

@Rohlik
Copy link
Contributor

Rohlik commented May 10, 2017

@teamfighter Internal hubot is integrated, so you don't need install anything after you install RC.

@ghost
Copy link

ghost commented May 10, 2017

@Rohlik
That s great, but why it is not working?

@wemu
Copy link

wemu commented May 10, 2017

is the bot invited to the channel? "bot help" does nothing? is the bot using a username that exists in the users list? does that user have proper permissions?
all I had to do was send a @rocket.cat hello to get the bot into the channel to interact with.

@ghost
Copy link

ghost commented May 10, 2017

@wemu
Thanks! It started working after I have invited bot to channel)

@Rohlik
Copy link
Contributor

Rohlik commented Sep 4, 2017

Is this already resolved with #7095 ?

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

No branches or pull requests

8 participants