Changeset e520e98b8f3e66fdfa978ec0f096baad8a7dfb38

Timestamp:

11/01/11 00:44:52 (2 years ago)

Author:

Theo Schlossnagle <jesus@lethargy.org>

git-committer:

Theo Schlossnagle <jesus@lethargy.org> 1320108292 -0700

git-parent:

[3c0b5cfefa3533bd9495cecbb70b5fd9d5250390], [895f8f377833cef214643a6ff35c402fd6835a25]

git-author:

Theo Schlossnagle <jesus@lethargy.org> 1320108292 -0700

Message:

Merge pull request #27 from Kami/include_body_http_check_option

Add 'include_body' option to the HTTP check

Files:

• src/modules-lua/noit/module/http.lua (modified) (4 diffs)

src/modules-lua/noit/module/http.lua

@@ -92 +92 @@
-               default="10000"
-               allowed="\d+">This sets the PCRE internal match limit (see pcreapi documentation).</parameter>
+    <parameter name="include_body"
+               required="optional"
+               allowed="^(?:true|false|on|off)$"
+               default="false"
+               allowed="\d+">Include whole response body as a metric with the key 'body'.</parameter>
+
-  </checkconfig>
-  <examples>
@@ -270 +276 @@
-    local pcre_match_limit = check.config.pcre_match_limit or 10000
-    local redirects = check.config.redirects or 0
+    local include_body = false
-
-    -- expect the worst
@@ -294 +301 @@
-    if schema == 'https' then
-        use_ssl = true
+    end
+
+    -- Include body as a metric
+    if check.config.include_body == "true" or check.config.include_body == "on" then
+        include_body = true
-    end
-
@@ -487 +499 @@
-    end
-
+    -- Include body
+    if include_body then
+        check.metric_string('body', output or '')
+    end
+
-    -- ssl ctx
-    local ssl_ctx = client:ssl_ctx()