fix: prohibit full names with a trailing `.`.

Just like in Git, it's considered invalid.

Sebastian Thiel committed 2 years ago

Commit 91d29c31 parent 9d5d8a64 files · +26 −10Browse files

Unified · Split

4 files

         StartsWithSlash,
         #[error("Multiple slashes in a row are not allowed as they may change the reference's meaning")]
         RepeatedSlash,
-        #[error("Names must not be a single '.', but may contain it.")]
+        #[error("Path components must not start with '.'")]
-        SingleDot,
+        StartsWithDot,
     }
     impl From<Infallible> for Error {
 use bstr::BStr;
-/// Validate a reference name running all the tests in the book. This disallows lower-case references, but allows
+/// Validate a reference name running all the tests in the book. This disallows lower-case references like `lower`, but also allows
-/// ones like `HEAD`.
+/// ones like `HEAD`, and `refs/lower`.
 pub fn name(path: &BStr) -> Result<&BStr, name::Error> {
     validate(path, Mode::Complete)
 }
         return Err(name::Error::StartsWithSlash);
     }
     let mut previous = 0;
-    let mut one_before_previous = 0;
     let mut saw_slash = false;
     for byte in path.iter() {
         match *byte {
-            b'/' if previous == b'.' && one_before_previous == b'/' => return Err(name::Error::SingleDot),
             b'/' if previous == b'/' => return Err(name::Error::RepeatedSlash),
+            b'.' if previous == b'/' => return Err(name::Error::StartsWithDot),
             _ => {}
         }
         if *byte == b'/' {
             saw_slash = true;
         }
-        one_before_previous = previous;
         previous = *byte;
     }

         Asterisk,
         #[error("A ref must not start with a '.'")]
         StartsWithDot,
+        #[error("A ref must not end with a '.'")]
+        EndsWithDot,
         #[error("A ref must not end with a '/'")]
         EndsWithSlash,
         #[error("A ref must not be empty")]
 }
 /// Assure the given `input` resemble a valid git tag name, which is returned unchanged on success.
+/// Tag names are provided as names, lik` v1.0` or `alpha-1`, without paths.
 pub fn name(input: &BStr) -> Result<&BStr, name::Error> {
     if input.is_empty() {
         return Err(name::Error::Empty);
     if input[0] == b'.' {
         return Err(name::Error::StartsWithDot);
     }
+    if input[input.len() - 1] == b'.' {
+        return Err(name::Error::EndsWithDot);
+    }
     if input.ends_with(b".lock") {
         return Err(name::Error::LockFileSuffix);
     }

         mktest!(
             refs_path_component_is_singular_dot,
             b"refs/./still-inside-but-not-cool",
-            RefError::SingleDot
+            RefError::StartsWithDot
         );
         mktest!(any_path_starts_with_slash, b"/etc/foo", RefError::StartsWithSlash);
         mktest!(empty_path, b"", RefError::Tag(TagError::Empty));
         mktest!(all_uppercase, b"MAIN");
         mktest!(all_uppercase_with_underscore, b"NEW_HEAD");
         mktest!(chinese_utf8, "refs/heads/你好吗".as_bytes());
+        mktest!(dot_in_directory_component, b"this./totally./works");
     }
     mod invalid {
             b".refs/somewhere",
             RefError::Tag(TagError::StartsWithDot)
         );
+        mktest!(
+            refs_path_name_starts_with_dot_in_name,
+            b"refs/.somewhere",
+            RefError::StartsWithDot
+        );
+        mktest!(
+            refs_path_name_ends_with_dot_in_name,
+            b"refs/somewhere.",
+            RefError::Tag(TagError::EndsWithDot)
+        );
         mktest!(
             refs_path_component_is_singular_dot,
             b"refs/./still-inside-but-not-cool",
-            RefError::SingleDot
+            RefError::StartsWithDot
         );
         mktest!(capitalized_name_without_path, b"Main", RefError::SomeLowercase);
         mktest!(lowercase_name_without_path, b"main", RefError::SomeLowercase);

         mktest!(contains_brackets, b"this_{is-fine}_too");
         mktest!(contains_brackets_and_at, b"this_{@is-fine@}_too");
         mktest!(dot_in_the_middle, b"token.other");
-        mktest!(dot_at_the_end, b"hello.");
         mktest!(slash_inbetween, b"hello/world");
     }
         mktestb!(contains_newline, b"prefix\nsuffix");
         mktestb!(contains_carriage_return, b"prefix\rsuffix");
         mktest!(starts_with_dot, b".with-dot", StartsWithDot);
+        mktest!(ends_with_dot, b"with-dot.", EndsWithDot);
         mktest!(empty, b"", Empty);
     }
 }