-
Notifications
You must be signed in to change notification settings - Fork 12
realm-server: HTTPS+HTTP/2 in local dev #4797
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
habdelra
wants to merge
72
commits into
main
Choose a base branch
from
worktree-cs-11114-http2-v2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
72 commits
Select commit
Hold shift + click to select a range
1655a6f
realm-server: speak HTTPS+HTTP/2 in local dev
habdelra 5de44d9
dispatcher + canonical URL migration for HTTPS flip
habdelra 01e3108
ensure-dev-cert soft-warn + env-vars conditional scheme
habdelra 1fa2d4b
HTTPS mandatory: provision dev cert in CI init, drop fallback paths
habdelra 55d518e
QUICKSTART: list mkcert prereq + dev-cert step
habdelra 5b383ee
test-services tasks: depend on ensure-dev-cert + scheme-aware readiness
habdelra 6e2e916
landing the review-agent + Copilot findings
habdelra 953f8c6
3 more review threads: README indent, dispatcher coverage, force-close
habdelra a5342d3
listener-dispatcher test: split logical-and assertions
habdelra caf7e7b
test harnesses: strip TLS env vars before spawning realm-server
habdelra 0a07028
testem-live: realm URL → https + --ignore-certificate-errors for Chrome
habdelra 3f3a76e
canonical-url migration: pre-check realm_user_permissions, not realm_…
habdelra fe5ee68
host environmentDefaults: keep test env on http
habdelra 88dc5bc
revert test-mode http revert; flip readiness scripts to https
habdelra 4d19e15
audit + flip http://localhost:42XX refs across the monorepo
habdelra 1b863c1
host testem.js: --ignore-certificate-errors for the Host Tests Chrome
habdelra b1f3cb9
runTestRealmServer*: strip TLS env vars in the shared helpers
habdelra b75edf1
Merge remote-tracking branch 'origin/main' into worktree-cs-11114-htt…
habdelra ec75fea
test scripts: flip stale http-get:// wait-on URLs to https-get://
habdelra 03b5a55
test fixtures: flip card adoptsFrom URLs to https for HTTPS realm-server
habdelra 2ef2397
diagnostic: bind realm-server as plain HTTPS+HTTP/1.1 via BOXEL_REALM…
habdelra aaf3b4c
yamllint: use double-quoted '1' in BOXEL_REALM_FORCE_HTTP1 env entries
habdelra f9e9955
host tests: flip http://localhost:42XX → https in test code
habdelra 20c0bba
ensure-dev-cert: init NSS DB before mkcert -install so Chromium trust…
habdelra 5e9704b
ensure-dev-cert: run NSS DB init + mkcert -install BEFORE the cert-skip
habdelra ff96ee4
ensure-dev-cert: log mkcert -install output + NSS DB contents
habdelra 4d84630
boxel-cli CI: wait for base realm 200 not 301 redirect
habdelra 7c4458d
diagnostic: bypass dispatcher with BOXEL_REALM_NO_DISPATCHER=1
habdelra b2507a6
revert: drop BOXEL_REALM_NO_DISPATCHER bypass — dispatcher isn't the …
habdelra 678e850
realm-server: fix HEAD-over-h2 hang by patching Koa response.writable
habdelra f491cb9
mise-tasks: realm-server-base / worker-base use https for --toUrl
habdelra 2fbebd9
host realm-indexing-test: move test/person URL to correct sort slot
habdelra 713e32c
realm-server tests: flip hardcoded http://localhost:4202/ to https
habdelra 7c6a88b
realm-server proxyAsset: strip h2 pseudo-headers before koa-proxies h…
habdelra 8bcc98a
realm-server proxyAsset: shadow req.headers instead of mutating it
habdelra a1298a7
realm-server proxyAsset: replace koa-proxies with a hand-rolled forwa…
habdelra 12b7fbc
matrix isolated-realm-server: strip TLS env vars from spawned children
habdelra 1cee373
Merge remote-tracking branch 'origin/main' into worktree-cs-11114-htt…
habdelra 442e278
ensure-dev-cert: fail fast when mkcert root CA isn't already trusted
habdelra b204160
dev-all: run ensure-dev-cert upfront so missing-cert errors are visible
habdelra 739da6b
dev-cert: add infra:trust-dev-cert task, fail fast with active-voice …
habdelra 1e1009a
ensure-dev-cert: name the exact command to re-run in the failure hint
habdelra 16e619a
vite: serve HTTPS at localhost:4200 when the dev cert is present
habdelra 495be71
vite: serve HTTPS at localhost:4200 when the dev cert is present
habdelra 852e6a1
matrix: migration script to flip user account_data URLs http→https
habdelra 6c2d7f8
vite: same-port http→https redirect dispatcher for local dev
habdelra a466e7f
prerender: warm vite over HTTPS in dev so the first standby doesn't t…
habdelra 4c138e7
Audit + flip http://localhost:4200 → https://localhost:4200 across th…
habdelra 77e7c69
env-vars: prefer system chrome over puppeteer's bundled chrome 143
habdelra c6e848b
Merge branch 'worktree-cs-11114-http2-v2' of https://github.com/cards…
habdelra 9a45b4d
realm-test-harness: strip TLS env vars before spawning vite preview
habdelra dfff3a0
fix experiments url
habdelra 9ce00a7
vite-with-traefik: prettier formatting
habdelra 73ac30c
CI: fix dispatcher byte-peek and wait-on TLS trust
habdelra 3399902
realm-server + CI: hardcode https serverURL, scope SF probe to icons-…
habdelra 438b82b
postgres ensure-db-exists: force TCP inside container, fail loud on C…
habdelra 190b73c
vite-with-traefik: skip the same-port redirect dispatcher for vite pr…
habdelra f5de14e
add reversible migrate-down for http↔https rewrite (postgres + matrix)
habdelra de5129a
prerender + standby probe: pair --ignore-certificate-errors with --al…
habdelra 26f5eac
matrix harness: switch isolated realm-server on :4205 to HTTPS+HTTP/2
habdelra 9ee116e
canonical-url migration: add :4205 to the http→https rewrite
habdelra 26acb01
workspace-sync-cli: switch test harness on :4205 to HTTPS
habdelra d5a6860
Merge remote-tracking branch 'origin/main' into worktree-cs-11114-htt…
habdelra 24693b9
host/matrix/realm-server test scripts: pin START_SERVER_AND_TEST_INSE…
habdelra 9706c34
wait-for-host-standby: add chrome-event logging to diagnose CI frame-…
habdelra 4ce991f
ci-host workflow: install dbus/upower BEFORE starting test services
habdelra 60360c1
testem: pair --ignore-certificate-errors with --allow-insecure-localhost
habdelra 1dc733a
host-submode-test: flip subdomain assertions to https
habdelra deee75a
post-review cleanup: cert + migration symmetry + dead-code removal
habdelra 3e1acdf
address Codex review feedback + tighten cert-relax gate
habdelra 7832e3e
listener-dispatcher tests: assert 308, not 301
habdelra 27035c6
vite-with-traefik: pin internal vite to 127.0.0.1 behind dispatcher (…
backspace File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has anyone actually tried following the steps in this document in a while? I too updated it with the mise changes but I’m not convinced it actually works. It doesn’t have to be addressed as part of this PR but I think we should consider removing it, or committing to making sure it works and stays working.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have not. I think @tintinthong made this originally, perhaps he has thoughts?