Skip to content

Commit

Permalink
Merge pull request #559 from nghialt/master
Browse files Browse the repository at this point in the history
Add HTTPClient interface to Pusher struct
  • Loading branch information
beorn7 authored May 5, 2019
2 parents 19d9302 + 6ea6f07 commit f1d50bc
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions prometheus/push/push.go
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,11 @@ import (

const contentTypeHeader = "Content-Type"

// HTTPDoer is an interface for the one method of http.Client that is used by Pusher
type HTTPDoer interface {
Do(*http.Request) (*http.Response, error)
}

// Pusher manages a push to the Pushgateway. Use New to create one, configure it
// with its methods, and finally use the Add or Push method to push.
type Pusher struct {
Expand All @@ -61,7 +66,7 @@ type Pusher struct {
gatherers prometheus.Gatherers
registerer prometheus.Registerer

client *http.Client
client HTTPDoer
useBasicAuth bool
username, password string

Expand Down Expand Up @@ -170,7 +175,11 @@ func (p *Pusher) Grouping(name, value string) *Pusher {

// Client sets a custom HTTP client for the Pusher. For convenience, this method
// returns a pointer to the Pusher itself.
func (p *Pusher) Client(c *http.Client) *Pusher {
// Pusher only needs one method of the custom HTTP client: Do(*http.Request).
// Thus, rather than requiring a fully fledged http.Client,
// the provided client only needs to implement the HTTPDoer interface.
// Since *http.Client naturally implements that interface, it can still be used normally.
func (p *Pusher) Client(c HTTPDoer) *Pusher {
p.client = c
return p
}
Expand Down

0 comments on commit f1d50bc

Please sign in to comment.