feat: throw error for invalid remotes

tanhauhau · tanhauhau · commit 40be69b50aed · 2021-11-28T18:08:25.000+08:00
diff --git a/lib/util/extractUrlAndGlobal.js b/lib/util/extractUrlAndGlobal.js
@@ -11,5 +11,8 @@
  */
 module.exports = function extractUrlAndGlobal(urlAndGlobal) {
 	const index = urlAndGlobal.indexOf("@");
+	if (index <= 0 || index === urlAndGlobal.length - 1) {
+		throw new Error(`Invalid request "${urlAndGlobal}"`);
+	}
 	return [urlAndGlobal.substring(index + 1), urlAndGlobal.substring(0, index)];
 };
diff --git a/test/extractUrlAndGlobal.unittest.js b/test/extractUrlAndGlobal.unittest.js
@@ -21,4 +21,15 @@ describe("extractUrlAndGlobal", () => {
 			"_"
 		]);
 	});
+	it("should throw error if starts with @", () => {
+		expect(() => extractUrlAndGlobal("@something")).toThrow();
+	});
+
+	it("should throw error if ends with @", () => {
+		expect(() => extractUrlAndGlobal("something@")).toThrow();
+	});
+
+	it("should throw error if do not have @", () => {
+		expect(() => extractUrlAndGlobal("something")).toThrow();
+	});
 });
