Skip to content

Authenticate Connect shares in the in-app Browser - #4132

Open
brsbl wants to merge 2 commits into
mainfrom
bb/bb-browser-closes-on-connect-share-links-thr_n92euqycph
Open

brsbl wants to merge 2 commits into
mainfrom
bb/bb-browser-closes-on-connect-share-links-thr_n92euqycph

Conversation

@brsbl

@brsbl brsbl commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

Human comments

What was wrong

The desktop Connect session was installed only in Electron's default session, while the in-app Browser uses the isolated persist:bb-browser session. Because Electron session partitions do not share cookies, Connect port-share requests from the Browser reached the 401 sign-in gate.

What changed

  • Installs and verifies each minted Connect cookie in both the main and personal Browser sessions.
  • Renews both session stores through the existing Connect renewal path.
  • Keeps the server-provided .getbb.app cookie domain, covering both brsbl.getbb.app and <host>--<port>.getbb.app hosts.

How you verified

  • Remote red proof: The regression saw zero Browser-session cookie writes before the fix.
  • Remote green proof: Full CI passed, including the new desktop regression test.
  • Desktop before/after evidence is blocked because this checkout runs on a headless Linux worker without an isolated desktop app. No bb Nightly, production app, or personal Browser was used.

BB-Thread-ID: thr_n92euqycph

AGENT GENERATED

This branch has not been deployed

No deployments
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