Skip to content

Commit

Permalink
Enhance HTTP connection options for phpfpm input plugin (influxdata#5713
Browse files Browse the repository at this point in the history
)
  • Loading branch information
glinton authored and bitcharmer committed Oct 18, 2019
1 parent ce31ec1 commit d98c1c1
Show file tree
Hide file tree
Showing 2 changed files with 37 additions and 4 deletions.
10 changes: 10 additions & 0 deletions plugins/inputs/phpfpm/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,16 @@ Get phpfpm stats using either HTTP status page or fpm socket.
## Example of multiple gathering from local socket and remote host
## urls = ["http://192.168.1.20/status", "/tmp/fpm.sock"]
urls = ["http://localhost/status"]

## Duration allowed to complete HTTP requests.
# timeout = "5s"

## Optional TLS Config
# tls_ca = "/etc/telegraf/ca.pem"
# tls_cert = "/etc/telegraf/cert.pem"
# tls_key = "/etc/telegraf/key.pem"
## Use TLS but skip chain & host verification
# insecure_skip_verify = false
```

When using `unixsocket`, you have to ensure that telegraf runs on same
Expand Down
31 changes: 27 additions & 4 deletions plugins/inputs/phpfpm/phpfpm.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ import (
"sync"

"github.com/influxdata/telegraf"
"github.com/influxdata/telegraf/internal"
"github.com/influxdata/telegraf/internal/tls"
"github.com/influxdata/telegraf/plugins/inputs"
)

Expand All @@ -35,7 +37,9 @@ type metric map[string]int64
type poolStat map[string]metric

type phpfpm struct {
Urls []string
Urls []string
Timeout internal.Duration
tls.ClientConfig

client *http.Client
}
Expand All @@ -58,9 +62,19 @@ var sampleConfig = `
## "fcgi://10.0.0.12:9000/status"
## "cgi://10.0.10.12:9001/status"
##
## Example of multiple gathering from local socket and remove host
## Example of multiple gathering from local socket and remote host
## urls = ["http://192.168.1.20/status", "/tmp/fpm.sock"]
urls = ["http://localhost/status"]
## Duration allowed to complete HTTP requests.
# timeout = "5s"
## Optional TLS Config
# tls_ca = "/etc/telegraf/ca.pem"
# tls_cert = "/etc/telegraf/cert.pem"
# tls_key = "/etc/telegraf/key.pem"
## Use TLS but skip chain & host verification
# insecure_skip_verify = false
`

func (r *phpfpm) SampleConfig() string {
Expand Down Expand Up @@ -96,8 +110,17 @@ func (g *phpfpm) Gather(acc telegraf.Accumulator) error {
// Request status page to get stat raw data and import it
func (g *phpfpm) gatherServer(addr string, acc telegraf.Accumulator) error {
if g.client == nil {
client := &http.Client{}
g.client = client
tlsCfg, err := g.ClientConfig.TLSConfig()
if err != nil {
return err
}
tr := &http.Transport{
TLSClientConfig: tlsCfg,
}
g.client = &http.Client{
Transport: tr,
Timeout: g.Timeout.Duration,
}
}

if strings.HasPrefix(addr, "http://") || strings.HasPrefix(addr, "https://") {
Expand Down

0 comments on commit d98c1c1

Please sign in to comment.