Skip to content
This repository has been archived by the owner on Nov 21, 2018. It is now read-only.

Adding NewRelic middleware #76

Merged
merged 5 commits into from
Dec 15, 2015
Merged

Adding NewRelic middleware #76

merged 5 commits into from
Dec 15, 2015

Conversation

roberthoner
Copy link
Contributor

#32

Usage:

class MyService < Rester::Service
  use Rester::Middleware::NewRelic
end

Shows up like so:
image

image

@roberthoner roberthoner added this to the 0.5.1 milestone Dec 11, 2015
kayvonghaffari
kayvonghaffari reviewed Dec 15, 2015
View reviewed changes
lib/rester/middleware/new_relic.rb
def _determine_method(request)
if request.object_chain.length.odd?
case request.request_method
when 'GET' then 'search'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't know you could do this (when.. then). Nice!

@kayvonghaffari
Copy link
Contributor

Looks good, and all 459 tests pass!

kayvonghaffari added a commit that referenced this pull request Dec 15, 2015
@kayvonghaffari
Merge pull request #76 from payout/issue/32
d9f3188 
Adding NewRelic middleware
@kayvonghaffari kayvonghaffari merged commit d9f3188 into master Dec 15, 2015
@roberthoner roberthoner deleted the issue/32 branch February 3, 2016 17:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@kayvonghaffari kayvonghaffari

Labels
None yet
Projects
None yet
Milestone
0.5.1
Development

Successfully merging this pull request may close these issues.
2 participants
@roberthoner @kayvonghaffari