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

Webhook integration doesn't send @all messages. #4060

Closed
ronyodil opened this issue Aug 19, 2016 · 3 comments · Fixed by #4228
Closed

Webhook integration doesn't send @all messages. #4060

ronyodil opened this issue Aug 19, 2016 · 3 comments · Fixed by #4228

Comments

@ronyodil
Copy link

Your Rocket.Chat version: 0.37.1 (problem appeared in 0.36.0).

When trying to send message with @ALL tag I'm getting '{"success":false}'. Single-user tags and common messages work as before.
Changes to 'mention-all' permission does not help. (#3703)

$ curl -X POST --data-urlencode 'payload={"text":"@all: test"}' $WEBHOOKURL
{"success":false}
$ curl -X POST --data-urlencode 'payload={"text":" test"}' $WEBHOOKURL                                            {"success":true}
@Syirrus
Copy link

Syirrus commented Aug 21, 2016

@MartinSchoeler make sure the permissions are correct (set in admin) for the user you are using the API with to send messages to everyone.

@ronyodil
Copy link
Author

I'm using Rocket.Cat for that. He is in Bots group. Also tried to add him to admins, moderators and room owners groups. No luck.
'mention-all' set to allow for everyone.

@vrooze
Copy link

vrooze commented Sep 1, 2016

I get this too.
I've been through all of the process as far as I can trace and all mentions work except @all you simply get a response of :
{ "success": false }

The query is simple:
payload:{"text": "Alert test @all"}

I was originally using a custom user but like the OP, I shifted to rocket.cat. I have given the bot group permissions to mention-all and it made no difference.

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

Successfully merging a pull request may close this issue.

6 participants