usps-support 0.2.53 → 0.2.55

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6ec8ac34f7499390a693ef659146f5eb3daa6b49da02852174256d6cd39ec62a
-  data.tar.gz: dd4b7baa3db26305c0e9637140db783d8bdcb79e6080f3928fa60adf80e465a0
+  metadata.gz: 2c0cf433c2d144a8f87a14bd3dcb7dea7715d0f567a2373ae94995d8f6bebf24
+  data.tar.gz: a219ef3451fd7afa40ece594f54119ea29923d981f53ce84b3213b7d0c1419d8
 SHA512:
-  metadata.gz: e88a8e2cc206895e6c2884c74f9a9ee038a4cf176eadbfb96c257f4871daf6fd11eb1dd96fc1e5b39879e433ee01a6fd1cfc213887524f8f39926dd33bfd26ec
-  data.tar.gz: f2b45c152f44e96a57821a02afbf7987401e33de96553ac36a1b7ef3b2e0999fe0b5ef970b0b9382a1bc0aecd8ba90ac11bf7e760d71d85ed25b177b919f4be3
+  metadata.gz: 52e39a4662db847d220472723875c1bb9f45eb97f01e8484969703e6ae946dee9872063186bf2d37b2fd400a3efdc594f4b2d77409eac38046ef63264f71d685
+  data.tar.gz: a6a4fad3524230632c11f668dd5b190b21f9010bb5ad7579cc7e295222bdd4b86679a5016328143c9801bf907063f57f6b03b0d7d9800d9740298886cc7121ed

data/app/controllers/usps/support/health_controller.rb

@@ -20,6 +20,11 @@ module Usps::Support
   # and route to it (`get 'health' => 'health#show'`). For a bespoke dependency, define a
   # #check_<name> in the subclass (e.g. wrapping #check_http(url)) and add <name> to CHECKS.
   #
+  # Responds 503 when any check fails, EXCEPT to callers that pass ?probe=1, which always get 200
+  # and are expected to judge health by the sentinel in the body. That exists so a failing
+  # dependency does not register as an application 5XX on the load balancer -- see #response_status.
+  # Point external string-matching probes at /health?probe=1; leave it off everywhere else.
+  #
   # Checks run on a background thread, not the request thread: a per-process refresher updates a
   # cached snapshot every health_refresh_interval seconds and #show serves that snapshot, so a slow
   # or wedged dependency can't tie up the Passenger worker pool. Each check also runs under
@@ -124,7 +129,7 @@ module Usps::Support
 
       response.set_header('Cache-Control', 'no-store')
       response.set_header('Retry-After', '30') unless healthy
-      render(plain: body, content_type: 'text/plain', status: healthy ? :ok : :service_unavailable)
+      render(plain: body, content_type: 'text/plain', status: response_status(healthy))
     end
 
     # Fallback for the (rare) case where not even the first refresh could produce a snapshot.
@@ -132,9 +137,26 @@ module Usps::Support
       response.set_header('Cache-Control', 'no-store')
       response.set_header('Retry-After', '30')
       render(plain: "HEALTHCHECK_FAIL #{app_name}\nsnapshot=unavailable\n",
-             content_type: 'text/plain', status: :service_unavailable)
+             content_type: 'text/plain', status: response_status(false))
     end
 
+    # True when the caller asked to be judged on the body alone (?probe=1).
+    def probe_mode? = params[:probe].present?
+
+    # A Route 53 HTTPS_STR_MATCH probe decides health by matching the sentinel in the body, so for it
+    # the status code says nothing the body has not already said. That redundancy is not free: the
+    # ALB counts a 503 on this path as an application 5XX, landing it in HTTPCode_Target_5XX_Count
+    # next to genuine user-facing errors. A slow dependency then pages on an endpoint no user ever
+    # touched (2026-08-05: 155 such 5XX across exams and merit-marks, all of them this endpoint
+    # answering the Route 53 checker, none of them a real request).
+    #
+    # Probe-mode callers therefore get 200 with the identical body -- still HEALTHCHECK_FAIL, still
+    # every per-check line, so the probe goes red exactly as before. Everyone else keeps the
+    # status-coded contract, which is what a human curl or a status-code-driven monitor expects.
+    # A crashed app or dead worker still yields the web server's own 5xx with no sentinel, so total
+    # failure remains detectable either way.
+    def response_status(healthy) = healthy || probe_mode? ? :ok : :service_unavailable
+
     # One round of checks + diagnostics, invoked by the monitor's runner (background thread, or the
     # inline cold/stale path). Wrapped in the Rails executor so ActiveRecord connections checked out
     # by the DB checks are returned even though this runs outside a request. Returns the render data.

data/db/hq/websites_schema.rb

@@ -74,7 +74,7 @@ ActiveRecord::Schema[8.1].define do
     t.index ["sqnumber"], name: "csq_index"
   end
 
-  create_table "students", id: :integer, charset: "latin1", comment: "dtb-Active eddept HQ800 students", options: "ENGINE=MyISAM", force: :cascade do |t|
+  create_table "students", id: :integer, charset: "latin1", comment: "dtb-Active eddept HQ800 students", options: "ENGINE=InnoDB", force: :cascade do |t|
     t.string "csl", limit: 1
     t.string "crsid", limit: 17
     t.string "start", limit: 6

data/lib/usps/support/version.rb

@@ -2,6 +2,6 @@
 
 module Usps
   module Support
-    VERSION = '0.2.53'
+    VERSION = '0.2.55'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: usps-support
 version: !ruby/object:Gem::Version
-  version: 0.2.53
+  version: 0.2.55
 platform: ruby
 authors:
 - Julian Fiander