fix: dispatch oauth callback close to GTK main thread

Same thread-safety issue as the safety timeout - the go func()
with time.Sleep runs wv.Close() on a goroutine, triggering GTK
widget operations off the main thread.

Replace go+Sleep with time.AfterFunc+glib.IdleAdd for the same
500ms delay but dispatched safely to the GTK main loop.

commit: 495353a727a0af9947892681d1db9fd759ae0b09
author: brice · 8mo ago · 2026-02-16 14:26 UTC
parent: 89f346b
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +9 -6

M internal/ui/coordinator/content.go +9 -6


@@ -1837,12 +1837,15 @@ func (c *ContentCoordinator) setupOAuthAutoClose(
1837	1837		Str("reason", reason).
1838	1838		Msg("oauth callback detected, closing")
1839	1839		requestCloseOnce.Do(func() {
1840		-	go func() {
1841		-	time.Sleep(500 * time.Millisecond)
1842		-	if wv != nil && !wv.IsDestroyed() {
1843		-	wv.Close()
1844		-	}
1845		-	}()
	1840	+	time.AfterFunc(500*time.Millisecond, func() {
	1841	+	cb := glib.SourceFunc(func(_ uintptr) bool {
	1842	+	if wv != nil && !wv.IsDestroyed() {
	1843	+	wv.Close()
	1844	+	}
	1845	+	return false
	1846	+	})
	1847	+	glib.IdleAdd(&cb, 0)
	1848	+	})
1846	1849		})
1847	1850		}
1848	1851


@@ -1837,12 +1837,15 @@ func (c *ContentCoordinator) setupOAuthAutoClose(
1837		Str("reason", reason).	1837		Str("reason", reason).
1838		Msg("oauth callback detected, closing")	1838		Msg("oauth callback detected, closing")
1839		requestCloseOnce.Do(func() {	1839		requestCloseOnce.Do(func() {
1840	-	go func() {	1840	+	time.AfterFunc(500*time.Millisecond, func() {
1841	-	time.Sleep(500 * time.Millisecond)	1841	+	cb := glib.SourceFunc(func(_ uintptr) bool {
1842	-	if wv != nil && !wv.IsDestroyed() {	1842	+	if wv != nil && !wv.IsDestroyed() {
1843	-	wv.Close()	1843	+	wv.Close()
1844	-	}	1844	+	}
1845	-	}()	1845	+	return false
			1846	+	})
			1847	+	glib.IdleAdd(&cb, 0)
			1848	+	})
1846		})	1849		})
1847		}	1850		}
1848			1851