Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding timeout to alert.post and http_post #1462

Merged
merged 3 commits into from
Oct 27, 2017

Commits on Oct 24, 2017

  1. Adding timeout to alert.post and http_post

    Adding timeout to alert.post and http_post nodes.  Currently
    http.DefaultClient is used, which blocks indefinitely when the endpoint
    accepts the connection but does not respond to the HTTP request.
    Switching to using a non-default http.Client with a timeout defined per
    node.
    sputnik13 committed Oct 24, 2017
    Configuration menu
    Copy the full SHA
    3564f30 View commit details
    Browse the repository at this point in the history
  2. modifying bufpool to provide closingBuffer

    bufpool provided buffers result in race condition when used with a
    buffer user that expects to take ownership of the buffer (like http
    client appears to).  Returning an extended version of Buffer that allows
    it to be closed, where the closing the buffer performs a Reset() on the
    buffer and returns it to the pool it came from.
    sputnik13 committed Oct 24, 2017
    Configuration menu
    Copy the full SHA
    bf4cc98 View commit details
    Browse the repository at this point in the history

Commits on Oct 26, 2017

  1. reverting bufpool

    sputnik13 committed Oct 26, 2017
    Configuration menu
    Copy the full SHA
    598fde6 View commit details
    Browse the repository at this point in the history