broidHQ / integrations

Connect your App to Multiple Messaging Channels with the W3C Open standard.

https://www.broid.ai

GNU Affero General Public License v3.0

745 stars 83 forks source link

Broid-Slack is not working anymore ? #141

Open fungiboletus opened 7 years ago

fungiboletus commented 7 years ago

Overview

What project is this issue for?: broid-slack
Is this a bug, security, feature request, or feedback:

Bug

Details

Slack is not working. The webhook payload format may have changed.

Steps to Reproduce

Start Broid with Slack enabled
Create a new Slack application
Go to Event subscriptions
Register a request URL 5. Unfocus the input, so Slack will test the URL

Observed

Broid-slack has an exception because payloadStr is undefined. It seems like the payload format has changed.

Expected

No exception, request URL validated.

killix commented 7 years ago

Thanks for the report. The issue is because we didn't handle the verification challenge flow as the integration the real-time API.

I quickly added the support of the flow. But reminder we support only this subscriptions: event_callback, interactive_message, slash_command

related https://github.com/broidHQ/integrations/pull/143