Skip to content

Commit

Permalink
Fix leaky tcp connections in phpfpm plugin
Browse files Browse the repository at this point in the history
closes #1089
  • Loading branch information
sparrc committed Apr 26, 2016
1 parent 671b40d commit 411d8d7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ based on _prefix_ in addition to globs. This means that a filter like
- [#1022](https://github.com/influxdata/telegraf/issues/1022): Dont error deb/rpm install on systemd errors.
- [#1078](https://github.com/influxdata/telegraf/issues/1078): Use default AWS credential chain.
- [#1070](https://github.com/influxdata/telegraf/issues/1070): SQL Server input. Fix datatype conversion.
- [#1089](https://github.com/influxdata/telegraf/issues/1089): Fix leaky TCP connections in phpfpm plugin.

## v0.12.1 [2016-04-14]

Expand Down
1 change: 1 addition & 0 deletions plugins/inputs/phpfpm/phpfpm.go
Original file line number Diff line number Diff line change
Expand Up @@ -184,6 +184,7 @@ func (g *phpfpm) gatherHttp(addr string, acc telegraf.Accumulator) error {
return fmt.Errorf("Unable to connect to phpfpm status page '%s': %v",
addr, err)
}
defer res.Body.Close()

if res.StatusCode != 200 {
return fmt.Errorf("Unable to get valid stat result from '%s': %v",
Expand Down

0 comments on commit 411d8d7

Please sign in to comment.