Project

General

Profile

Actions

Bug #6943

closed

[Git hosting] arvados-git-httpd should return 4xx, not 5xx, for authentication errors

Added by Tom Clegg over 8 years ago. Updated over 8 years ago.

Status:
Duplicate
Priority:
Normal
Assigned To:
-
Category:
Git hosting
Target version:
-
Story points:
0.5

Description

Currently, as verified by tests, arvados-git-httpd returns 500 when the servers are all working perfectly but the client used an invalid token.

Existing test case:

func (s *IntegrationSuite) TestInvalidToken(c *check.C) {
    for _, repo := range []string{"active/foo.git", "active/foo/.git"} {
        err := s.runGit(c, "no-such-token-in-the-system", "fetch", repo)
        c.Assert(err, check.ErrorMatches, `.* 500 while accessing.*`)
    }
}

This response should have been 401, not 500.


Related issues

Is duplicate of Arvados - Bug #6780: [git] "git clone https://..." with a bogus token should respond 4xx, not 5xxNewActions
Actions #1

Updated by Brett Smith over 8 years ago

  • Status changed from New to Duplicate
  • Target version deleted (Arvados Future Sprints)

Duplicates #6780. I'll merge the descriptions.

Actions #2

Updated by Brett Smith over 8 years ago

  • Target version set to Arvados Future Sprints
Actions #3

Updated by Brett Smith over 8 years ago

  • Target version deleted (Arvados Future Sprints)
Actions

Also available in: Atom PDF