Skip to content

fix: disable oauth notification monitor when endpoint is unavailable#387

Open
Pnkcaht wants to merge 1 commit intodocker:mainfrom
Pnkcaht:fix/oauth-notify-reconnect-loop
Open

fix: disable oauth notification monitor when endpoint is unavailable#387
Pnkcaht wants to merge 1 commit intodocker:mainfrom
Pnkcaht:fix/oauth-notify-reconnect-loop

Conversation

@Pnkcaht
Copy link
Contributor

@Pnkcaht Pnkcaht commented Feb 6, 2026

What I did

  • Added a one-time availability check before starting the OAuth notification monitor.
  • If the endpoint is unavailable, the monitor does not start and logs only once.
  • Normal behavior is preserved on Docker Desktop.

Related Issue

Diagram

flowchart TD
    A[Start OAuth Monitor] --> B{Endpoint available?}
    B -- Yes --> C[Start monitor goroutine]
    B -- No --> D[Disable monitor, log once]
    C --> E[Connect to OAuth SSE stream]
    E --> F[Process events]
    D --> G[No goroutine, no spam]
Loading

Signed-off-by: pnkcaht <samzoovsk19@gmail.com>
@Pnkcaht Pnkcaht requested a review from a team as a code owner February 6, 2026 01:27
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.

Connecting to OAuth notification error SPAM IN TERMINAL

1 participant