Skip to content
This repository has been archived by the owner on Dec 11, 2023. It is now read-only.

Support proxy #112

Merged
merged 1 commit into from Dec 2, 2015
Merged

Support proxy #112

merged 1 commit into from Dec 2, 2015

Conversation

winebarrel
Copy link
Contributor

I added support for the http proxy.

Librato::Metrics.authenticate '...', '...'
Librato::Metrics.proxy 'http://proxy.example.com:8080'
Librato::Metrics.submit :my_metric => 42, :my_other_metric => 1002

Please merge If there is no problem.

@nextmat
Copy link
Contributor

nextmat commented Jul 14, 2015

Thanks for the PR, will get this reviewed shortly!

nextmat added a commit that referenced this pull request Dec 2, 2015
@nextmat
Merge pull request #112 from winebarrel/support-proxy
0bb57b7 
Support proxy
@nextmat nextmat merged commit 0bb57b7 into librato:master Dec 2, 2015
@nextmat
Copy link
Contributor

nextmat commented Dec 2, 2015

Apologies for the delays on this, released in v1.6.0

@winebarrel
Copy link
Contributor Author

Thank you! 😃

@nextmat nextmat mentioned this pull request Dec 2, 2015
Closed
@nextmat
Copy link
Contributor

nextmat commented Dec 2, 2015

Thank you!

My pleasure, sorry it took so long.

If it is of interest, I'm planning to percolate proxy support up into librato-rack and librato-rails. You may want to watch librato/librato-rack#45 if that matters to you.

@jderrett jderrett mentioned this pull request Mar 9, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.

None yet
2 participants
@winebarrel @nextmat