mirror of
https://github.com/garrytan/gstack.git
synced 2026-11-12 03:30:28 +01:00
fix: pre-landing review fixes
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
This commit is contained in:
1 parent
992fb32993
commit
6d0d2691d5
2 files changed
+4
-3
No files matched your search
+3
-2
@@ -1332,8 +1332,9 @@ profile, `state save` files and, with `BROWSE_PERSIST_STATE=1`, the headless
|
||||
session state), and prefer Aside for signed-in work where you have it.
|
||||
|
||||
To check a build, run `$B stop`, then the same `goto` in the default headless
|
||||
mode with `BROWSE_EXTENSIONS_DIR` unset, `BROWSE_PROXY_URL=direct` (so no proxy
|
||||
from `HTTPS_PROXY`, `HTTP_PROXY` or `ALL_PROXY` is used) and
|
||||
mode with `BROWSE_EXTENSIONS_DIR` and `BROWSE_PROXY_URL` unset, the proxy
|
||||
variables `HTTPS_PROXY`, `HTTP_PROXY` and `ALL_PROXY` unset in both cases
|
||||
(Chromium reads those itself, even when gstack sets no proxy), and
|
||||
`GSTACK_CHROMIUM_PATH` unset (the bundled build) or set to the build you want
|
||||
to check. First confirm that a known-good HTTPS site such as
|
||||
`$B goto https://example.com/` loads in the same session, because a captive
|
||||
|
||||
+1
-1
@@ -5,7 +5,7 @@
|
||||
**BROWSER.md now says that gstack's own browser still trusts the compromised eMudhra intermediates, and how to check (#3338).** Chrome blocked the eight eMudhra intermediate CAs with a CRLSet push on 2026-10-17, the day their key compromise was reported, and scheduled the removal of their root, emSign ECC Root CA - G3, from the Chrome Root Store for 2026-10-27. gstack's fallback browser gets neither in time: its default headless browser never receives CRLSet updates, headed mode and `handoff` received them only about a minute after each launch in testing on Linux (#3338, from macOS, reports that gstack's profile never gets one), and the root store compiled into the bundled Chromium still trusts the root. On 2026-10-24, `$B goto https://dv-g3b-emudhra.we-have-a-problem.com/` returned 200 and "That's bad!" on gstack's pinned Playwright 1.62.1 (Chromium 151.0.7922.34). The same check against Playwright 1.64.1 (Chromium 156.0.8078.4) and the 1.65.0 alpha of 2026-10-24 (Chromium 157.0.8103.15) also loaded the page. Chrome for Testing builds from 158.0.8115.0 fail it with `ERR_CERT_AUTHORITY_INVALID`, so a Playwright bump has to wait for a release that bundles one of those builds.
|
||||
|
||||
### Changed
|
||||
- **`BROWSER.md`**: a new "TLS trust: the bundled Chromium's root store" section, at the end of the security stack, says the fallback engine checks certificates against that compiled root store, that the headless browser never gets CRLSet updates while headed mode and `handoff` got them about a minute after launch in testing on Linux, and that the bundled build does not enforce Certificate Transparency. It names the commands that put real sessions and credentials into the browser (among them `cookie-import-browser`, `cookie-import`, `cookie`, `header`, `state load`, `storage set`, `fill`/`type`, `upload`, `handoff`, and a paired agent using `fill`/`type`), lists where imported sessions persist, says how `BROWSE_EXTENSIONS_DIR` and `GSTACK_CHROMIUM_PATH` change the picture (`handoff` always uses the bundled build, and a current Chrome only helps if it drops the root), and gives the `$B goto` check: after `$B stop`, in headless mode with `BROWSE_PROXY_URL=direct`, and once a known-good HTTPS site loads in the same session, only `ERR_CERT_AUTHORITY_INVALID` on the test host means a build is fixed. The table of contents, the fallback-engine introduction and the `cookie-import-browser` and `handoff` command rows link to it. Aside is unaffected, because it is your own browser with its own updates.
|
||||
- **`BROWSER.md`**: a new "TLS trust: the bundled Chromium's root store" section, at the end of the security stack, says the fallback engine checks certificates against that compiled root store, that the headless browser never gets CRLSet updates while headed mode and `handoff` got them about a minute after launch in testing on Linux, and that the bundled build does not enforce Certificate Transparency. It names the commands that put real sessions and credentials into the browser (among them `cookie-import-browser`, `cookie-import`, `cookie`, `header`, `state load`, `storage set`, `fill`/`type`, `upload`, `handoff`, and a paired agent using `fill`/`type`), lists where imported sessions persist, says how `BROWSE_EXTENSIONS_DIR` and `GSTACK_CHROMIUM_PATH` change the picture (`handoff` always uses the bundled build, and a current Chrome only helps if it drops the root), and gives the `$B goto` check: after `$B stop`, in headless mode with no proxy variables set, and once a known-good HTTPS site loads in the same session, only `ERR_CERT_AUTHORITY_INVALID` on the test host means a build is fixed. The table of contents, the fallback-engine introduction and the `cookie-import-browser` and `handoff` command rows link to it. Aside is unaffected, because it is your own browser with its own updates.
|
||||
|
||||
### For contributors
|
||||
- The edits above the cookie workflow excerpt change no line count, so the `setup-browser-cookies/SKILL.md workflow` judge input is byte-identical (prompt SHA-256 `c851e635…`, 12,805 bytes, before and after).
|
||||
|
||||
Reference in new issue
Block a user