From d48712de5b71579b3ff785adad023f4b17175ef5 Mon Sep 17 00:00:00 2001 From: Joey Hess Date: Tue, 31 Mar 2015 15:47:59 -0400 Subject: forgot to add this comment --- ...comment_1_73721f7709b1d8ca90522b2147428c6d._comment | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) create mode 100644 doc/bugs/Uses_suggested_by_CHECKURL-CONTENTS_filename_instead_of_explicitly_specified_one_in_addurl_--file_/comment_1_73721f7709b1d8ca90522b2147428c6d._comment (limited to 'doc') diff --git a/doc/bugs/Uses_suggested_by_CHECKURL-CONTENTS_filename_instead_of_explicitly_specified_one_in_addurl_--file_/comment_1_73721f7709b1d8ca90522b2147428c6d._comment b/doc/bugs/Uses_suggested_by_CHECKURL-CONTENTS_filename_instead_of_explicitly_specified_one_in_addurl_--file_/comment_1_73721f7709b1d8ca90522b2147428c6d._comment new file mode 100644 index 000000000..76bd70453 --- /dev/null +++ b/doc/bugs/Uses_suggested_by_CHECKURL-CONTENTS_filename_instead_of_explicitly_specified_one_in_addurl_--file_/comment_1_73721f7709b1d8ca90522b2147428c6d._comment @@ -0,0 +1,18 @@ +[[!comment format=mdwn + username="joey" + subject="""comment 1""" + date="2015-03-31T18:48:12Z" + content=""" +I think I didn't fully consider the `addurl --file` behavior when adding this +feature. + +It seems to make sense for CLAIMURL to be used, since a special remote may +claim the url. It seems to make sense for CHECKURL to then be called (if +not --relaxed) to check that the url's content is present before adding it. + +If CHECKURL-CONTENTS returns a file, it makes sense to check the file +size, but not use the suggested filename. + +If CHECKURL-MULTI returns multiple files, I think it makes sense to give up +with an error. +"""]] -- cgit v1.2.3