Skip to content

Comments

fix: health checks no longer modify endpoint updated_at timestamp#212

Merged
IonesioJunior merged 1 commit intomainfrom
fix-healthcheck-preserve-updated-at
Feb 18, 2026
Merged

fix: health checks no longer modify endpoint updated_at timestamp#212
IonesioJunior merged 1 commit intomainfrom
fix-healthcheck-preserve-updated-at

Conversation

@IonesioJunior
Copy link
Member

@IonesioJunior IonesioJunior commented Feb 18, 2026

Summary

  • Health monitor status updates now preserve the endpoint's updated_at timestamp
  • Previously, every health check cycle triggered SQLAlchemy's onupdate hook, causing all health-checked endpoints to have their updated_at refreshed
  • This caused endpoints from users with many endpoints (e.g., cambridge-press-oa with 177) to dominate public listings sorted by updated_at DESC
  • Fix: Explicitly set updated_at=EndpointModel.updated_at in the atomic UPDATE statement, bypassing the ORM's onupdate hook

Test plan

  • Verify health checks run successfully without errors
  • Confirm endpoints from various users appear in public listings (not just users with many endpoints)
  • Check that updated_at only changes when users actually update their endpoints

@IonesioJunior IonesioJunior merged commit 57d2e81 into main Feb 18, 2026
21 checks passed
@IonesioJunior IonesioJunior deleted the fix-healthcheck-preserve-updated-at branch February 18, 2026 17:28
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.

1 participant