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

[http_check] check response for content to http_check #1390

Merged
merged 5 commits into from
Mar 13, 2015

Conversation

yannmh
Copy link
Member

@yannmh yannmh commented Feb 23, 2015

  • Add an optional content_match parameter to look for a particular string within the HTTP response.

Rebase #1297 and #1326 on 5.2.1 agent release.

Added a few changes:

  • Add an optional http_response_status_code parameter to look for a specific HTTP response status code (Fixes http_check - define success #1403)
  • Rebased tests on AgentCheckTest
  • Adding tests
  • Cosmetic changes

Thanks @handigarde @chrissnell

@yannmh yannmh self-assigned this Feb 23, 2015
@yannmh yannmh force-pushed the yann/http_regexp_match branch 7 times, most recently from a76cc29 to ad7dfb8 Compare February 27, 2015 20:43
@remh remh added this to the 5.3.0 milestone Mar 2, 2015
@yannmh yannmh force-pushed the yann/http_regexp_match branch from ad7dfb8 to d561ce3 Compare March 6, 2015 22:15
@yannmh yannmh force-pushed the yann/http_regexp_match branch from d561ce3 to 9e86409 Compare March 6, 2015 22:25
yannmh added a commit that referenced this pull request Mar 13, 2015
[http_check] check response for content to http_check
@yannmh yannmh merged commit 8fc200f into master Mar 13, 2015
@yannmh yannmh deleted the yann/http_regexp_match branch March 14, 2015 18:02
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 9e86409 on yann/http_regexp_match into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 9e86409 on yann/http_regexp_match into * on master*.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

http_check - define success
5 participants