scottk/chatwoot
1
0
Fork 0
mirror of https://github.com/lingble/chatwoot.git synced 2025-11-02 12:08:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: ActiveRecord::RecordNotFound Couldn't find Channel::WebWidget (#6523)

Browse Source 
* fix: resucue ActiveRecord::RecordNotFound for webwidget controller

* chore: add rails.log
This commit is contained in:
Vishnu Narayanan
2023-02-23 19:16:07 +05:30
committed by GitHub
parent 6407745571
commit 71f2b27728
2 changed files with 11 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/controllers/widgets_controller.rb
View File

@@ -18,6 +18,9 @@ class WidgetsController < ActionController::Base
def set_web_widget def set_web_widget
@web_widget = ::Channel::WebWidget.find_by!(website_token: permitted_params[:website_token]) @web_widget = ::Channel::WebWidget.find_by!(website_token: permitted_params[:website_token])
rescue ActiveRecord::RecordNotFound
Rails.logger.error('web widget does not exist')
render json: { error: 'web widget does not exist' }, status: :not_found
end end
def set_token def set_token

8
spec/controllers/widgets_controller_spec.rb
View File

@@ -33,5 +33,13 @@ describe '/widget', type: :request do
expect(response).to have_http_status(:unauthorized) expect(response).to have_http_status(:unauthorized)
expect(response.body).to include('Account is suspended') expect(response.body).to include('Account is suspended')
end end
it 'returns 404 if the webwidget is deleted' do
web_widget.delete
get widget_url(website_token: web_widget.website_token)
expect(response).to have_http_status(:not_found)
expect(response.body).to include('web widget does not exist')
end
end end
end end