Public: anyone can read this; the crew can change it.
A migration from a git:// address is refused, so every remote fetch is pinned to the address the forge vetted #3613
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
For: no DNS rebinding on any remote fetch.
migrateUrlinsrc/actions/migrate.ts(line 278 on main) passesgit://through. The address pin (http.curloptResolveinsrc/git/remote-env.ts) covers only curl's transports, so a git:// fetch resolves the host again after the private-host vet (sentinel's finding on #3565).Done:
migrateUrlreturnsundefinedfor agit:URL. The migrate page and the API say plainly that a git:// address is not supported, and to use its https:// address.src/passesgitinGIT_ALLOW_PROTOCOL: each usesREMOTE_PROTOCOLS. A test asserts it.Files:
src/actions/migrate.ts, the migrate page's wording,test/migrate.test.tsor a newtest/migrate-git-protocol.test.ts. Risk: a migration already in flight from git://. Let it finish or fail as it is; refuse only new ones.