aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoseph Hunkeler <jhunkeler@gmail.com>2025-06-27 12:41:52 -0400
committerJoseph Hunkeler <jhunkeler@gmail.com>2025-06-27 12:41:52 -0400
commit2425ef83ac7c36947ee1119399b1e4155649ed80 (patch)
treed20c23757cb68f2b8d7a23edb49af9f6ca14cc75
parent54a3824b06c80e5e6bbe11a5dc102de85f47f15a (diff)
downloadstasis-bug-fixes-again.tar.gz
A note to selfbug-fixes-again
-rw-r--r--src/lib/delivery/delivery.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/lib/delivery/delivery.c b/src/lib/delivery/delivery.c
index 5c2767c..7ec2e04 100644
--- a/src/lib/delivery/delivery.c
+++ b/src/lib/delivery/delivery.c
@@ -241,6 +241,9 @@ void delivery_defer_packages(struct Delivery *ctx, int type) {
}
// Is the list item a git+schema:// URL?
+ // TODO: nametmp is just the name so this will never work. but do we want it to? this looks like
+ // TODO: an unsafe feature. We shouldn't be able to change what's in the config. we should
+ // TODO: be getting what we asked for, or exit the program with an error.
if (strstr(nametmp, "git+") && strstr(nametmp, "://")) {
char *xrepo = strstr(nametmp, "+");
if (xrepo) {