CalvinBackup/anonymous_github
1
0
Fork 0
mirror of https://github.com/tdurieux/anonymous_github.git synced 2026-09-12 13:48:58 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: archive legacy repository IDs containing spaces

Browse Source
This commit is contained in:
tdurieux
2026-09-08 14:08:48 +02:00
parent 530e388e46
commit 90f14b19b7
2 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/recover-repository-owners.test.js
+4 -4
View File
@@ -176,7 +176,7 @@ describe("archive ownerless repositories", () => {
expect(f.writes).to.have.length(0);
});
it("rejects unsafe storage paths before changing the record", async () => {
for (const repoId of [undefined, "", ".", "..", "../other", "a/b", "/etc"]) {
for (const repoId of [undefined, "", " ", ".", "..", ".. ", "../other", "a/b", "a\\b", "/etc", "repo\n", "repo\0"]) {
const f = fixture([{ _id: "repo", repoId }]);
await recoverRepositoryOwners(f.db, { ...f.options, apply: true, archiveAllOwnerless: true });
expect(f.events[0].issue).to.equal("unsafe_or_missing_repo_id");
@@ -255,14 +255,14 @@ describe("archived repository access", () => {
const before = config.FOLDER;
try {
config.FOLDER = root;
for (const id of ["archive-target", "keep-sibling"]) {
for (const id of ["Paccmann Polymer", "keep-sibling"]) {
fs.mkdirSync(path.join(root, id, "original"), { recursive: true });
fs.writeFileSync(path.join(root, id, "original", "file.txt"), "cached content");
}
const f = fixture([{ _id: "repo", repoId: "archive-target" }]);
const f = fixture([{ _id: "repo", repoId: "Paccmann Polymer" }]);
await recoverRepositoryOwners(f.db, { ...f.options, apply: true, archiveAllOwnerless: true,
deleteCache: id => new FileSystem().rm(id) });
expect(fs.existsSync(path.join(root, "archive-target", "original"))).to.equal(false);
expect(fs.existsSync(path.join(root, "Paccmann Polymer", "original"))).to.equal(false);
expect(fs.existsSync(path.join(root, "keep-sibling", "original", "file.txt"))).to.equal(true);
expect(f.writes).to.have.length(2);
} finally { config.FOLDER = before; fs.rmSync(root, { recursive: true, force: true }); }