diff options
| author | Adam Soltys <[email protected]> | 2026-08-15 11:51:18 -0700 |
|---|---|---|
| committer | Adam Soltys <[email protected]> | 2026-08-15 11:51:18 -0700 |
| commit | 89ee463e94727d0bb605d956cd5c61a33a426a4a (patch) | |
| tree | 3165417946be9a5a5e30406aadda81500bffefc0 | |
| parent | 2e75736f5b3173023e81e3dc29ab6eeb24710d84 (diff) | |
| download | coinosv3-89ee463e94727d0bb605d956cd5c61a33a426a4a.tar.gz coinosv3-89ee463e94727d0bb605d956cd5c61a33a426a4a.zip | |
A downed notifier no longer vetoes new connections — only the user's own denial does; self-hosted builds create tab-answering connections and arm background when they can
Co-Authored-By: Claude Fable 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_014tE4U5nEjGTQy8PwYqamfv
| -rw-r--r-- | src/features/nwc.js | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/features/nwc.js b/src/features/nwc.js index 90ccb84..ef34373 100644 --- a/src/features/nwc.js +++ b/src/features/nwc.js @@ -638,10 +638,13 @@ export function nwcFeature(ctx) { try { await enableBackground(); } catch (e) { - // Denied at the prompt (or dismissed it) — anything else is a real - // failure whose message is more useful than a permissions lecture. + // Denied at the prompt (or dismissed it): that's the user's call and + // the connection is refused. But permission GRANTED with the plumbing + // failing (notifier down, a self-hosted build with no notifier at all) + // must not hold the connection hostage — it still answers from open + // tabs, and ensureBackground keeps retrying the arming on every start. if (Notification.permission !== 'granted') throw new Error(t('nwcNotifNeeded')); - throw e; + console.warn('nwc: background arming failed, creating the connection anyway:', e.message); } const s = load(); delete s.backgroundOff; save(s); } |
