Skip to content

Fix 'Missing :controller key on routes definition'#11

Open
inclooder wants to merge 1 commit into
Netflix:masterfrom
inclooder:fix_missing_controller_keyword_in_routes
Open

Fix 'Missing :controller key on routes definition'#11
inclooder wants to merge 1 commit into
Netflix:masterfrom
inclooder:fix_missing_controller_keyword_in_routes

Conversation

@inclooder

@inclooder inclooder commented Mar 22, 2018 •
edited
Loading

Copy link
Copy Markdown

I have the following error after running rake middleware. I use rails 5.2.1 and ruby 2.5.0

rake aborted!                                                                                                                                                                                                                                                                     
ArgumentError: Missing :controller key on routes definition, please check your routes.                                                                                                                                                                                            
/bundler_data/gems/newrelic_rpm-3.18.1.330/lib/new_relic/agent/instrumentation/rake.rb:28:in `invoke'                                                                                                                                                                             
/bundler_data/gems/rake-12.3.0/exe/rake:27:in `<top (required)>'                                                                                                                                                                                                                  
Tasks: TOP => middleware => environment                                                                                                                                                                                                                                           
(See full trace by running task with --trace)

Fixed it by adding to: to routes with 404 response.
We should never hit this but it's necessary to satisfy rails :)

@kuiro5

kuiro5 commented Apr 12, 2018

Copy link
Copy Markdown

+1

@kirantpatil

kirantpatil commented Jan 29, 2019

Copy link
Copy Markdown

Any updates on this MR, why it is still not merged ?

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@inclooder @kuiro5 @kirantpatil