mirror of
https://github.com/lingble/chatwoot.git
synced 2025-11-02 20:18:08 +00:00
29158e32fe
- Add a warning logger for cases where we are getting webhook events for inactive numbers. - Add config to discard events for inactive numbers so that the meta will stop sending events --------- Co-authored-by: Pranav <pranav@chatwoot.com>
63 lines
2.6 KiB
Ruby
63 lines
2.6 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe 'Webhooks::WhatsappController', type: :request do
|
|
let(:channel) { create(:channel_whatsapp, provider: 'whatsapp_cloud', sync_templates: false, validate_provider_config: false) }
|
|
|
|
describe 'GET /webhooks/verify' do
|
|
it 'returns 401 when valid params are not present' do
|
|
get "/webhooks/whatsapp/#{channel.phone_number}"
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
|
|
it 'returns 401 when invalid params' do
|
|
get "/webhooks/whatsapp/#{channel.phone_number}",
|
|
params: { 'hub.challenge' => '123456', 'hub.mode' => 'subscribe', 'hub.verify_token' => 'invalid' }
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
|
|
it 'returns challenge when valid params' do
|
|
get "/webhooks/whatsapp/#{channel.phone_number}",
|
|
params: { 'hub.challenge' => '123456', 'hub.mode' => 'subscribe', 'hub.verify_token' => channel.provider_config['webhook_verify_token'] }
|
|
expect(response.body).to include '123456'
|
|
end
|
|
end
|
|
|
|
describe 'POST /webhooks/whatsapp/{:phone_number}' do
|
|
it 'call the whatsapp events job with the params' do
|
|
allow(Webhooks::WhatsappEventsJob).to receive(:perform_later)
|
|
expect(Webhooks::WhatsappEventsJob).to receive(:perform_later)
|
|
post '/webhooks/whatsapp/123221321', params: { content: 'hello' }
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
|
|
context 'when phone number is in inactive list' do
|
|
before do
|
|
allow(GlobalConfig).to receive(:get_value).with('INACTIVE_WHATSAPP_NUMBERS').and_return('+1234567890,+9876543210')
|
|
end
|
|
|
|
it 'returns service unavailable for inactive phone number in URL params' do
|
|
allow(Rails.logger).to receive(:warn)
|
|
expect(Rails.logger).to receive(:warn).with('Rejected webhook for inactive WhatsApp number: +1234567890')
|
|
|
|
post '/webhooks/whatsapp/+1234567890', params: { content: 'hello' }
|
|
expect(response).to have_http_status(:unprocessable_entity)
|
|
expect(response.parsed_body['error']).to eq('Inactive WhatsApp number')
|
|
end
|
|
end
|
|
|
|
context 'when INACTIVE_WHATSAPP_NUMBERS config is not set' do
|
|
before do
|
|
allow(GlobalConfig).to receive(:get_value).with('INACTIVE_WHATSAPP_NUMBERS').and_return(nil)
|
|
end
|
|
|
|
it 'processes the webhook normally' do
|
|
allow(Webhooks::WhatsappEventsJob).to receive(:perform_later)
|
|
expect(Webhooks::WhatsappEventsJob).to receive(:perform_later)
|
|
|
|
post '/webhooks/whatsapp/+1234567890', params: { content: 'hello' }
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
end
|
|
end
|
|
end
|