Antero Mejr <m...@antr.me> writes: > +(define (validate-substitute-url url-str) > + (if (string->uri url-str) > + url-str > + (error "Not a valid substitute URL: " url-str))) > +
> + (substitute-urls > + (map validate-substitute-url > + (append (guix-extension-substitute-urls extension) > + (guix-configuration-substitute-urls config)))) Should we instead create a validate-substitute-urls and use that as a sanitizer for the guix-extension and guix-configuration records? This would catch errors during record creation instead of service creation, as well as still perform validation if anything else does or will use those records in the future. -- Take it easy, Richard Sent Making my computer weirder one commit at a time.
