rbot-remote: allow override of function

fix: webservice dispatch syntax check

Early bailout if the command field is not specified in the POST request.

(Not doing this actually leads to an exception in the subsequent debug
because command is an Array and it can't be added to a String,
but ayway the early catch is cleaner error handling.)

Also add a TODO about a potential improvement for permission error handling.

fix: webservice message user type

When the user logs in, m.source is already a botuser.

Partial revert: reintroduce rbot-remote

It now interfaces with the webservice 'dispatch' feature
instead of the previous remotectl plugin, but the external API
remains the same, for compatibility with existing users.

webhook: define number for watch/star actions too

When a star is added/removed, we get notified with a message
that has no number, title or link. The pseudo-sentence is also
very bare (X started/created/deleted, without an object).

Show the number of watchers and/or stars (as appropriate) to produce
a more meaningful phrase.

bot: reorder fork sequence

We do the actual fork as the last step, after redirecting the
standard I/O streams. This also ensures that no log messages
are shown on console _after_ forking.

logger: don't actually close the console logger

We nil the internal variable, but we should not close the logger itself,
since this leads to STDERR being closed and the interpreter reopening it
for its own needs, or something like that, which ultimately leads to
the interpreter's warning appearing in console (probably due to ruby
itself reopening stderr to be able to output the messages).

If we simply nil the variable, the interpreter is content with sending
us the messages per our shenanigangs, and everything works just as
expected.

fix: write override behavior

IO.write() takes an arbitrary number of argumens, that are converted to string
and joined on write. We should behave the same way.

Moreover, the returned value is the number of bytes, and this is bytesize
in modern Ruby.

fix: restore logging of session start/end

logger: fix STDERR recursion

Overriding (as we should do) the standard error write method when
daemonizing introduces a recursion in the logger, due to the
persistent presence of the console logger.

The solution is obviously to stop logging on console
after daemonization.

fix: stderr vs stdout write override

In 96d13521d the STDOUT/STDERR write overrides were replaced
by overrides to $stdout, but the second should have been to $stderr
(obviously).

chucknorris: fix loading

There was an extra read.

webhook: include target ref for pull/merge requests

Also add a TODO about a possible event-based output filter.

webhook: typo preventing unwatch

webhook: gitlab support

new plugin: webhook

This plugin provides webhook support for GitHub and similar services,
through the webservice feature. Hosting services and output formats
can be customized through filters as usual.

plugin(rss): fix config description

fix: restart logger thread after fork

Logging was broken when daemonizing, due to the logger thread being dead
after the fork. This can be solved by restarting the thread, if necessary
when setting the log file (which we conveniently do right after the fork).

fix: preserve loglevels across options and files

Catch another non-standard error

Properly quote salut-italian

chucknorris: read gzip stream before passing it to YAML.load

Prevents errors about external_encoding not being defined for the gzip
stream in ruby 1.9+

quiz: hint presentation fix for ruby 1.9+

Merge pull request #61 from ruby-rbot/dependabot/bundler/nokogiri-1.11.4

build(deps): bump nokogiri from 1.11.1 to 1.11.4

build(deps): bump nokogiri from 1.11.1 to 1.11.4

Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.11.1 to 1.11.4.
- [Release notes](https://github.com/sparklemotion/nokogiri/releases)
- [Changelog](https://github.com/sparklemotion/nokogiri/blob/main/CHANGELOG.md)
- [Commits](https://github.com/sparklemotion/nokogiri/compare/v1.11.1...v1.11.4)

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #59 from ruby-rbot/dependabot/bundler/mechanize-2.7.7

build(deps): bump mechanize from 2.7.6 to 2.7.7

Merge pull request #60 from ruby-rbot/dependabot/bundler/oauth-0.5.5

build(deps-dev): bump oauth from 0.5.4 to 0.5.5

build(deps-dev): bump oauth from 0.5.4 to 0.5.5

Bumps [oauth](https://github.com/oauth-xx/oauth-ruby) from 0.5.4 to 0.5.5.
- [Release notes](https://github.com/oauth-xx/oauth-ruby/releases)
- [Changelog](https://github.com/oauth-xx/oauth-ruby/blob/master/HISTORY)
- [Commits](https://github.com/oauth-xx/oauth-ruby/compare/v0.5.4...v0.5.5)

Signed-off-by: dependabot[bot] <support@github.com>

build(deps): bump mechanize from 2.7.6 to 2.7.7

Bumps [mechanize](https://github.com/sparklemotion/mechanize) from 2.7.6 to 2.7.7.
- [Release notes](https://github.com/sparklemotion/mechanize/releases)
- [Changelog](https://github.com/sparklemotion/mechanize/blob/master/CHANGELOG.rdoc)
- [Commits](https://github.com/sparklemotion/mechanize/compare/v2.7.6...v2.7.7)

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #52 from ruby-rbot/dependabot/bundler/rack-2.2.3

build(deps): bump rack from 2.2.2 to 2.2.3

build(deps): bump rack from 2.2.2 to 2.2.3

Bumps [rack](https://github.com/rack/rack) from 2.2.2 to 2.2.3.
- [Release notes](https://github.com/rack/rack/releases)
- [Changelog](https://github.com/rack/rack/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rack/rack/compare/v2.2.2...2.2.3)

Signed-off-by: dependabot[bot] <support@github.com>

plugin(imdb): fix title regex, closes #44

refactor: remove global bot instance, closes #42

plugin(keywords): remove export to file, see #42

plugin(factoids): use registry for storage see #42

ensures the path reported by gems does exists

plugin(quotes): use registry for storage see #42

plugin(alias): use registry for storage see #42

plugin(lart): fix not save before load

plugin(salut): use registry for storage see #42

plugin(lart): refactor to use registry to persist

see #42

core: sets plugin_path to loaded plugins

While loading a plugin in the manager using #load_botmodule_file this
will remember the directory of the file that is being loaded in
@next_plugin_path. In the plugin/BotModule initialization we set the
path in plugin_path of the plugin.

This was the only solution I could find to get this information in the
plugin. It is useful because this allows the plugin to know in which
directory it is located, it can then easily access data files stored
alongside the plugin.

Some built-in plugins (see #42) use the data/rbot/templates to copy
data files on bot load (see #repopulate_botclass_directory) into the
profile directory (~/.rbot/).

refactor: wordlist shouldn't use bot singleton #35

also related to #41 and #6

refactor: remove unused userdata module

I never saw a plugin use this feature besides wof, IRC user objects should not hold
persistent data like this, I rather have the IRC interface clean. The
general idea is a good though, especially easier per-channel, per-user
data persistence.

plugin(wheelfortune): remove botdata dependency

plugin(translator): removed google translate

yandex still works surprisingly enough

fix nick renaming during connection

refactor: httputil no longer core module see #38

This is intended to make testing/mocking of the http client easier.

httputil: remove obsolete version_1_2 declaration

remove obsolete require of net/https

readme: added build status image

plugin(oxford): fix empty service result see #37

plugin(points): forgot one special case, see #34

Tokens that include itself ++/-- are now ignored as well.

plugin(points): new message parser, see #34

message: add #thanks method, similar to okay

plugin(oxford): fix result handling, closes #37

plugin(points): disregard self after stripping color codes

plugin(points): strip/ignore some terms closes #34

- strip color/bold codes in terms
- ignore terms with ++/-- in them

plugin(hangman): fixes word generator closes #9

This now uses wordgenerator.net

plugin(oxford): moved to lexico.com, closes #13

fix: in-memory registry persist correctly

plugin(note): test cases added, closes #24

plugin(points): +/- must come after, closes #34

This modifies the karma/points plugin to ignore increment/
decrement suffixes. `--SOMETHING` is more trouble than its worth,
people will write --NAME as a signature, or paste a command
line argument, e.g. `ls --sort time` which causes issues.

I also added tests for the points plugin, the plan is
to make the plugin testing easier more "rubionic"

tests: mocked bot and messages added, added points tests

registry: add in-memory implementation for tests

fix: gettext updated each_textdomain > each_text_domains

plugin(search): fix wolfram and gdef, removed some

this removes gcount and gtime from the search plugin,
google no longer provides this information easily

plugin(search): fix search and gcalc, closes #28, #29

plugin(rot13): test assert added

test: rot13 simplified

refactor: logger moved away from ircbot (exp.)

This moves the logger management thread/queue to a seperate singleton.
It removes the explicit stopping/restarting of the logging thread since
the thread should behave like a daemon thread anyway.
Still needs to be tested to work in daemonize.

added simple dockerfile

test: fix shadowed test method

test: add gettext/unittest to gemfile

test: change db adapter to dbm

test: first plugin test added for rot13

test: optional registry adapter tests

ci: fix rake invocation in script

fix: plugins delegate disregarding all options given

test: small changes and fixes to existing tests

ci: add rake test command to script

added rake and updated Gemfile and Gemfile.lock

plugin(imdb): changed base url

ci: fix travis ci config

always log to stdout with debug flag

This makes it so the -d debug flag on the rbot
command line will always log to STDOUT, previously
it would just force the log level.

ci: add system dependencies to travis ci

plugin(script): remove deprecated $SAFE

closes #32

experiment with travis-ci setup

gem: support for newest ruby versions

deps: update dependencies

Merge pull request #30 from ruby-rbot/dependabot/bundler/nokogiri-1.10.9

Bump nokogiri from 1.6.1 to 1.10.9

Merge pull request #31 from ruby-rbot/dependabot/bundler/rack-1.6.13

Bump rack from 1.5.2 to 1.6.13

Bump rack from 1.5.2 to 1.6.13

Bumps [rack](https://github.com/rack/rack) from 1.5.2 to 1.6.13.
- [Release notes](https://github.com/rack/rack/releases)
- [Changelog](https://github.com/rack/rack/blob/master/CHANGELOG.md)
- [Commits](https://github.com/rack/rack/compare/1.5.2...1.6.13)

Signed-off-by: dependabot[bot] <support@github.com>

Bump nokogiri from 1.6.1 to 1.10.9

Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.6.1 to 1.10.9.
- [Release notes](https://github.com/sparklemotion/nokogiri/releases)
- [Changelog](https://github.com/sparklemotion/nokogiri/blob/master/CHANGELOG.md)
- [Commits](https://github.com/sparklemotion/nokogiri/compare/v1.6.1...v1.10.9)

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #2 from mroch/patch-1

Update markov.rb

Merge pull request #4 from ahpook/rename_karma

Rename and improve karma plugin

Merge pull request #27 from ohadlevy/gemspec-fixes

this enables to build again from gemspec

this enables to build again from gemspec

fixes registry for 2.3.0, bug caused by wrong #get_impl

previously getimpl returned eigenclasses aswell which
always was an issue but i guess a minor change in ruby
triggered this.

Improvements to the points plugin

This commit makes a couple of improvements the points plugin:
- added a 'pointsdump' command to show everything in the registry,
  not just the best and worst
- fixed Hash#index deprecation warnings under ruby >1.9
- made nicer messages when displaying stats on an empty registry