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

Update node-irc under slack-irc-bridge #115

Closed
1 of 2 tasks
darkk opened this issue Jun 28, 2017 · 3 comments
Closed
1 of 2 tasks

Update node-irc under slack-irc-bridge #115

darkk opened this issue Jun 28, 2017 · 3 comments

Comments

@darkk
Copy link
Contributor

darkk commented Jun 28, 2017

Used docker container seems to have old node-irc module that has a bug disrespecting messageSplit that was fixed in martynsmith/node-irc#385.

It leads to messages truncated in the middle of word & loosing ~20 symbols while relaying large message from slack to IRC. That's quite annoying :(

Also, Slack bot messages likely do not propagate to IRC anymore, see hellais fork for the feature.

  • messageSplit
  • Slack bot -> IRC (for monitoring) (2018-08-30: we don't rely on IRC logs of #ooni-bots anymore)
@darkk darkk self-assigned this Jun 28, 2017
@hellais
Copy link
Member

hellais commented Jul 25, 2017

Another neat feature to add to the slack bot, is maybe the ability to relay messages using a custom nick depending on the users slack handle.

I believe it's possible to send messages to an IRC channel without joining it, but I am not sure if you can set the nick without issuing a /nick command. If it is possible we could support sending /msg to the #ooni- channels with the name $USERNAME-slack, making the IRC usability a bit better.

@darkk
Copy link
Contributor Author

darkk commented Jul 25, 2017

I am not sure if you can set the nick without issuing a /nick command

AFAIK, you can't. Outgoing PRIVMSG has no nickname.

I use weechat with gateway_rename.scm plugin and plugins.var.guile.gateway_rename.gateways = "(OFTC * slacktopus <NICK>) setting, it works for me, but it's not a generic solution.

@darkk darkk removed their assignment Mar 30, 2019
@hellais
Copy link
Member

hellais commented Nov 19, 2019

This is done

@hellais hellais closed this as completed Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants