On 09-09-12 11:30 PM, Junio C Hamano wrote: >> @@ -5196,8 +5202,9 @@ sub git_snapshot { >> die_error(403, "Unsupported snapshot format"); >> } >> >> - if (!defined $hash) { >> - $hash = git_get_head_hash($project); >> + my $full_hash = git_get_full_hash($project, $hash); >> + if (!$full_hash) { >> + die_error(404, 'Hash id was not valid'); >> } > > This is in the context of "snapshot", so obviously you care more about > just "such an object exists", don't you? You also want it to be a > tree-ish. Try giving it $hash = 'junio-gpg-pub' and see how it breaks. You have confused me. How is using 'junio-gpg-pub' different from the second test case that tries to use 'frizzumFrazzum'? -- Mark Rada (ferrous26) marada@xxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html