[Bug]: Health erroneously checks Stored Procedures

[JerryNixon]

What?

Health check returns check results for stored procs. It should ONLY include tables and views.

Health output sample

{
  "status": "Healthy",
  "version": "1.7.81",
  "app-name": "dab_oss_1.7.81",
  "timestamp": "2025-11-17T20:33:42.2752261Z",
  "configuration": {
    "rest": true,
    "graphql": true,
    "mcp": true,
    "caching": true,
    "telemetry": false,
    "mode": "Development"
  },
  "checks": [
    {
      "status": "Healthy",
      "name": "MSSQL",
      "tags": [
        "data-source"
      ],
      "data": {
        "response-ms": 3,
        "threshold-ms": 1000
      }
    },
    {
      "status": "Healthy",
      "name": "GetSeriesActors", // stored procedure
      "tags": [
        "graphql",
        "endpoint"
      ],
      "data": {
        "response-ms": 1,
        "threshold-ms": 1000
      }
    },
    {
      "status": "Healthy",
      "name": "GetSeriesActors", // stored procedure
      "tags": [
        "rest",
        "endpoint"
      ],
      "data": {
        "response-ms": 5,
        "threshold-ms": 1000
      }
    }
  ]
}

[souvikghosh04]

@JerryNixon / @Aniruddh25 should stored procedures and functions be discarded from health checks permanently?

[JerryNixon]

The entity checks in the Health endpoint check every table and view type entity with a user-configurable select with a first compared against a user-configurable threshold. We do not check stored procedures, and cannot check stored procedures, as we do not have any mechanism to take parameters as Health configuration values. Also stored procedures are not guaranteed to be deterministic, making checks that would call them potentially be unreliable. So, yes, stored procedures should be ignored.