threads / bug / 39455

BUG: Wrong error message when access to .git directory denied

Subject: BUG: Wrong error message when access to .git directory denied

## tl;dr

4 messages between May 29, 2015 and May 29, 2015.

replies: 3people: 3as markdown or json

Jonas Berlin· May 29, 2015, 07:47 UTC · lore

Hi,

Severity: Low
Impact: Confusion
Git version: 1.9.1

If I don't have permissions to access the .git folder of a non-bare git repository, I get the following output:

  fatal: Not a git repository (or any of the parent directories): .git

Expected output:

  fatal: Permission denied while accessing .git directory

Steps to reproduce:

  mkdir foo
  cd foo
  git init
  chmod 000 .git
  git log

- xkr47

Junio C Hamano· May 29, 2015, 15:29 UTC · re: Jonas Berlin · lore

Re: BUG: Wrong error message when access to .git directory denied

Jonas Berlin <xkr47@outerspace.dyndns.org> writes:

> Severity: Low
> Impact: Confusion

;-)

Show 10 quoted lines

> Git version: 1.9.1
>
> If I don't have permissions to access the .git folder of a non-bare
> git repository, I get the following output:
>
>   fatal: Not a git repository (or any of the parent directories): .git
>
> Expected output:
>
>   fatal: Permission denied while accessing .git directory

But the thing is, .git being unreadable simply means we do not know if it is a .git directory, or a random directory that does not have anything to do with Git that happens to have that name.

So "Not a git repository" is the best Git can say.

Stefan Beller· May 29, 2015, 16:45 UTC · re: Junio C Hamano · lore

Re: BUG: Wrong error message when access to .git directory denied

On Fri, May 29, 2015 at 8:29 AM, Junio C Hamano <gitster@pobox.com> wrote:

Show 23 quoted lines

> Jonas Berlin <xkr47@outerspace.dyndns.org> writes:
>
>> Severity: Low
>> Impact: Confusion
>
> ;-)
>
>> Git version: 1.9.1
>>
>> If I don't have permissions to access the .git folder of a non-bare
>> git repository, I get the following output:
>>
>>   fatal: Not a git repository (or any of the parent directories): .git
>>
>> Expected output:
>>
>>   fatal: Permission denied while accessing .git directory
>
> But the thing is, .git being unreadable simply means we do not know
> if it is a .git directory, or a random directory that does not have
> anything to do with Git that happens to have that name.
>
> So "Not a git repository" is the best Git can say.

fatal: Permission denied while accessing .git directory

does not imply we're having a git repository here at all, but rather sounds like "We wanted to check for you if this is a git repository, but we cannot do that as we lack permissions to do so.

So git doesn't know if it is inside a repo or not, so the actual error message "Not a git repository" is wrong. It's just that the current permissions cannot tell if we're in a repository or not.

> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Junio C Hamano· May 29, 2015, 17:06 UTC · re: Stefan Beller · lore

Re: BUG: Wrong error message when access to .git directory denied

Stefan Beller <sbeller@google.com> writes:

Show 11 quoted lines

>> So "Not a git repository" is the best Git can say.
>
> fatal: Permission denied while accessing .git directory
>
> does not imply we're having a git repository here at all, but rather
> sounds like "We wanted to check for you if this is a git repository,
> but we cannot do that as we lack permissions to do so.
>
> So git doesn't know if it is inside a repo or not, so the actual error message
> "Not a git repository" is wrong. It's just that the current permissions cannot
> tell if we're in a repository or not.

Fair enough.  Patches ;-)?