search

supabase / realtime

Broadcast, Presence, and Postgres Changes via WebSockets
https://supabase.com/realtime
Apache License 2.0
6.83k stars 315 forks source link

fix: Shutdown Connect when migrations fail #1151

Closed filipecabaco closed 2 months ago

filipecabaco commented 2 months ago

What kind of change does this PR introduce?

Currently we had an issue where Connect module was not shutting down properly leading to more and more connections to be created against the tenant database. Now we use reason :shutdown to enforce the process is killed.

Also the existing Listen/Notify approach will be deleted for a different approach to be built in the future.

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **realtime-demo** | ⬜️ Ignored ([Inspect](https://vercel.com/supabase/realtime-demo/ESzvkySSS1YVbe1WgJK997M35Hdo)) | [Visit Preview](https://realtime-demo-git-fix-shutdown-connect-module-supabase.vercel.app) | | Sep 11, 2024 4:17pm |
kiwicopple commented 2 months ago

:tada: This PR is included in version 2.32.7 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: