deployangel 0.1.8 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 26338624984d9d36012cf0e66b2448ddf31985ae9b629c4ca211e467c5e39e6c
-  data.tar.gz: bbbb651d96b7091563b263e20df94bc8471bf132bbf8a0511297176c111a74ed
+  metadata.gz: e2bc6936b563039a1c2caa91f3a8125f0c32582366505ab35dfabf2ec90af3c5
+  data.tar.gz: 22c805b0adef58922b3513afc19492433421358be2e75fd12513e650a7e57c9c
 SHA512:
-  metadata.gz: 5846a9913395c0defeeb41b1e5f5ad20a924439ee92ca899b09ae76dcbf935748432cf2c02e01b81772029796f39f5cc221c690313b30ea73a2831aac11825d0
-  data.tar.gz: 534d6218818cb03b8be1f4144d21d129484b112acdb08a18c53300f62c67dd7198d9dc48a66284ada08069ea08a0795f9cce705d8c6baa11f4fc371d5edf88e5
+  metadata.gz: 361b042f1d885fc63442dff6d4f4c406b36fd14ea5c226cf04a5bea2f7537770c6b4f4ad169331b95e6f3bdacf37169f90ac70fc6567006203f4884fe021236a
+  data.tar.gz: 328b709591c7c5441390fb74dc535d0a515061fd64161bbe555ffe9f4f8602bf32f10676b6119458e44837ee539e96e5e2f4d72b97cafc05e203d3ef2eb77e7b

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.1.9 (2026-10-04)
+
+- A 4xx response to a request no route matched, such as a bot probing
+  `/wp-admin` or `.env`, or middleware turning a request away before
+  routing, is still recorded under `GET unmatched` but no longer counts in
+  the app's request count, status counts, or latency. It made apps look
+  busier than their real pages, and fast 404s diluted their latency. An
+  unrouted 5xx still counts.
+
 ## 0.1.8 (2026-10-04)
 
 - Health checks served by a lambda or a mounted Rack app at a conventional

data/lib/deployangel/agent.rb

@@ -42,12 +42,12 @@ module DeployAngel
       @active
     end
 
-    def record_request(route_key:, status:, duration_ms:, unhandled: false)
+    def record_request(route_key:, status:, duration_ms:, unhandled: false, in_totals: true)
       return unless @active
 
       after_fork! if Process.pid != @pid
       start_reporter
-      @aggregator.record(route_key: route_key, status: status, duration_ms: duration_ms, unhandled: unhandled)
+      @aggregator.record(route_key: route_key, status: status, duration_ms: duration_ms, unhandled: unhandled, in_totals: in_totals)
     rescue StandardError => e
       warn_once(:record, "DeployAngel failed to record a request: #{e.class}: #{e.message}")
     end

data/lib/deployangel/core/aggregator.rb

@@ -89,11 +89,13 @@ module DeployAngel
         (@job_classes[key] ||= JobStats.empty).record_discard
       end
 
-      def record(route_key, status, duration_ms, unhandled, max_routes)
-        @requests += 1
-        @status_counts[status.to_s] += 1 if status >= 400
-        @unhandled_exceptions += 1 if unhandled
-        @histogram.record(duration_ms)
+      def record(route_key, status, duration_ms, unhandled, max_routes, in_totals)
+        if in_totals
+          @requests += 1
+          @status_counts[status.to_s] += 1 if status >= 400
+          @unhandled_exceptions += 1 if unhandled
+          @histogram.record(duration_ms)
+        end
 
         key = @routes.key?(route_key) || @routes.size < max_routes - 1 ? route_key : OTHER_ROUTE
         route = (@routes[key] ||= RouteStats.empty)
@@ -112,11 +114,13 @@ module DeployAngel
       @mutex = Mutex.new
     end
 
-    def record(route_key:, status:, duration_ms:, unhandled: false)
+    # in_totals: false records the request under its route only, leaving it
+    # out of the app-wide counts and latency.
+    def record(route_key:, status:, duration_ms:, unhandled: false, in_totals: true)
       started_at = period_start(@clock.call)
       @mutex.synchronize do
         (@periods[started_at] ||= Period.new(started_at))
-          .record(route_key, status.to_i, duration_ms, unhandled, @max_routes)
+          .record(route_key, status.to_i, duration_ms, unhandled, @max_routes, in_totals)
       end
     end
 

data/lib/deployangel/rails/http.rb

@@ -45,7 +45,7 @@ module DeployAngel
           route = route_key(env)
           return if route && DeployAngel.configuration.ignored_route?(route)
           # Unrouted successes are static files and similar; unrouted
-          # errors (such as routing 404s) still count.
+          # errors (such as routing 404s) are still recorded.
           return if route.nil? && status.to_i < 400
 
           key = route || "#{env["REQUEST_METHOD"]} unmatched"
@@ -54,7 +54,13 @@ module DeployAngel
             route_key: key,
             status: status.to_i,
             duration_ms: (Process.clock_gettime(Process::CLOCK_MONOTONIC) - started) * 1000.0,
-            unhandled: unhandled
+            unhandled: unhandled,
+            # A 4xx no route matched is mostly bots probing paths like
+            # /wp-admin, or middleware turning requests away. It stays
+            # visible under "unmatched", but out of the app's totals, so it
+            # doesn't add to the evidence or dilute real pages' latency. An
+            # unrouted 5xx still counts: something broke.
+            in_totals: !route.nil? || status.to_i >= 500
           )
         rescue StandardError
           nil

data/lib/deployangel/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DeployAngel
-  VERSION = "0.1.8"
+  VERSION = "0.1.9"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: deployangel
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.1.9
 platform: ruby
 authors:
 - Jordan Owens