Return 200 if the hinted repository is not found

This is contrary to what we should be logically doing but Gitlab
considers hooks returning non-2xx as failed. It should be fine to say
"we don't know about this repository" via HTTP status codes, but c'est
la vie, I guess.

This plus [0] are a recipe for disaster. Anyway, even if [0] was not
the case disabling the hook for 10 minutes is still unacceptable for
Jens.

The workaround is to return 2xx, even in this case.

[0] https://gitlab.com/gitlab-org/gitlab/-/issues/387938

Author: nbarrientos

jens/webapps/gitlabproducer.py

@@ -62,7 +62,7 @@ def hello_gitlab():
         return 'Queue not accessible', 500
     except NameError as error:
         logging.error("'%s' couldn't be found in repositories" % (url))
-        return 'Repository not found', 404
+        return 'Repository not found', 200
     except Exception as error:
         logging.error("Unexpected error (%s)" % repr(error))
         return 'Internal Server Error!', 500

jens/webapps/test/test_gitlabproducer.py

@@ -83,7 +83,7 @@ def test_repository_not_found(self):
                          'git_ssh_url': "file://foo"
                         }
                     }))
-        self.assertEqual(reply.status_code, 404)
+        self.assertEqual(reply.status_code, 200)
 
     @patch('jens.webapps.gitlabproducer.enqueue_hint')
     def test_secret_token_ignored_if_not_configured(self, mock_eq):