|
Patrick Uiterwijk |
a50651 |
# -*- coding: utf-8 -*-
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
"""
|
|
Patrick Uiterwijk |
a50651 |
(c) 2015-2018 - Copyright Red Hat Inc
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
Authors:
|
|
Patrick Uiterwijk |
a50651 |
Patrick Uiterwijk <puiterwijk@redhat.com></puiterwijk@redhat.com>
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
"""
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
from __future__ import unicode_literals
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
__requires__ = ['SQLAlchemy >= 0.8']
|
|
Patrick Uiterwijk |
a50651 |
import pkg_resources
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
import datetime
|
|
Patrick Uiterwijk |
a50651 |
import unittest
|
|
Patrick Uiterwijk |
a50651 |
import shutil
|
|
Patrick Uiterwijk |
a50651 |
import sys
|
|
Patrick Uiterwijk |
a50651 |
import tempfile
|
|
Patrick Uiterwijk |
a50651 |
import os
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
import six
|
|
Patrick Uiterwijk |
a50651 |
import json
|
|
Patrick Uiterwijk |
a50651 |
import pygit2
|
|
Patrick Uiterwijk |
a50651 |
from mock import patch, MagicMock
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
sys.path.insert(0, os.path.join(os.path.dirname(
|
|
Patrick Uiterwijk |
a50651 |
os.path.abspath(__file__)), '..'))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
import pagure.lib
|
|
Patrick Uiterwijk |
a50651 |
import tests
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
class PagureFlaskAppClonetests(tests.Modeltests):
|
|
Patrick Uiterwijk |
a50651 |
""" Tests for the clone bridging. """
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
def setUp(self):
|
|
Patrick Uiterwijk |
a50651 |
super(PagureFlaskAppClonetests, self).setUp()
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
tests.create_projects(self.session)
|
|
Patrick Uiterwijk |
a50651 |
tests.create_tokens(self.session)
|
|
Patrick Uiterwijk |
a50651 |
tests.create_tokens_acl(self.session)
|
|
Patrick Uiterwijk |
a50651 |
self.create_project_full('clonetest', {"create_readme": "y"})
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {'ALLOW_HTTP_PULL_PUSH': False})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_clone_disabled(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that the HTTP clone endpoint gets correctly closed. """
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/clonetest.git/info/refs?service=git-upload-pack')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 403)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('not allowed', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {'ALLOW_HTTP_PULL_PUSH': True})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_clone_invalid_service(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that the HTTP endpoint refuses invalid services. """
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/clonetest.git/info/refs?service=myservice')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 400)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('Unknown service', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {'ALLOW_HTTP_PULL_PUSH': True})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_clone_invalid_project(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that the HTTP endpoint refuses invalid projects. """
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/nosuchrepo.git/info/refs?service=git-upload-pack')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 404)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('Project not found', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {'ALLOW_HTTP_PULL_PUSH': True})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_clone_dumb(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that the HTTP endpoint refuses dumb service request. """
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/clonetest.git/info/refs')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 400)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('Please switch', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PULL_PUSH': True,
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PUSH': False,
|
|
Patrick Uiterwijk |
a50651 |
'HTTP_REPO_ACCESS_GITOLITE': None,
|
|
Patrick Uiterwijk |
a50651 |
})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_push_disabled(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that the HTTP push gets refused. """
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/clonetest.git/info/refs?service=git-receive-pack')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 403)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('pushing disabled', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.post('/clonetest.git/git-receive-pack')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 403)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('pushing disabled', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PULL_PUSH': True,
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PUSH': True,
|
|
Patrick Uiterwijk |
a50651 |
'HTTP_REPO_ACCESS_GITOLITE': None,
|
|
Patrick Uiterwijk |
a50651 |
})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_push_unauthed(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that the HTTP push gets refused unauthed. """
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/clonetest.git/info/refs?service=git-receive-pack')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 403)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('Unauthenticated push', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {'ALLOW_HTTP_PULL_PUSH': True})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_clone_private_project_unauthed(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that the HTTP endpoint enforced project.private. """
|
|
Patrick Uiterwijk |
a50651 |
project = pagure.lib._get_project(self.session, 'clonetest')
|
|
Patrick Uiterwijk |
a50651 |
project.private = True
|
|
Patrick Uiterwijk |
a50651 |
self.session.add(project)
|
|
Patrick Uiterwijk |
a50651 |
self.session.commit()
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/clonetest.git/info/refs?service=git-upload-pack')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 404)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('Project not found', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PULL_PUSH': True,
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PUSH': False,
|
|
Patrick Uiterwijk |
a50651 |
'HTTP_REPO_ACCESS_GITOLITE': None,
|
|
Patrick Uiterwijk |
a50651 |
})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_clone(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that HTTP cloning gives reasonable output. """
|
|
Patrick Uiterwijk |
a50651 |
# Unfortunately, actually testing a git clone would need the app to
|
|
Patrick Uiterwijk |
a50651 |
# run on a TCP port, which the test environment doesn't do.
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/clonetest.git/info/refs?service=git-upload-pack')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 200)
|
|
Patrick Uiterwijk |
a50651 |
output_text = output.get_data(as_text=True)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn("# service=git-upload-pack", output_text)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn(" refs/heads/master\n0000", output_text)
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
802c7e |
output = self.app.post(
|
|
Patrick Uiterwijk |
802c7e |
'/clonetest.git/git-upload-pack',
|
|
Patrick Uiterwijk |
802c7e |
headers={'Content-Type': 'application/x-git-upload-pack-request'},
|
|
Patrick Uiterwijk |
802c7e |
)
|
|
Patrick Uiterwijk |
f607ef |
# Git 2.17 returns 415, older return 200
|
|
Patrick Uiterwijk |
f607ef |
# Either means we didn't fully crash when returning the response
|
|
Patrick Uiterwijk |
f607ef |
self.assertIn(output.status_code, (200, 415))
|
|
Patrick Uiterwijk |
802c7e |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PULL_PUSH': True,
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PUSH': False,
|
|
Patrick Uiterwijk |
a50651 |
'HTTP_REPO_ACCESS_GITOLITE': None,
|
|
Patrick Uiterwijk |
a50651 |
})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_clone_private(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that HTTP cloning gives reasonable output with project.private. """
|
|
Patrick Uiterwijk |
a50651 |
# Unfortunately, actually testing a git clone would need the app to
|
|
Patrick Uiterwijk |
a50651 |
# run on a TCP port, which the test environment doesn't do.
|
|
Patrick Uiterwijk |
a50651 |
project = pagure.lib._get_project(self.session, 'clonetest')
|
|
Patrick Uiterwijk |
a50651 |
project.private = True
|
|
Patrick Uiterwijk |
a50651 |
self.session.add(project)
|
|
Patrick Uiterwijk |
a50651 |
self.session.commit()
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get('/clonetest.git/info/refs?service=git-upload-pack')
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 404)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn('Project not found', output.get_data(as_text=True))
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get(
|
|
Patrick Uiterwijk |
a50651 |
'/clonetest.git/info/refs?service=git-upload-pack',
|
|
Patrick Uiterwijk |
a50651 |
environ_overrides={'REMOTE_USER': 'pingou'},
|
|
Patrick Uiterwijk |
a50651 |
)
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 200)
|
|
Patrick Uiterwijk |
a50651 |
output_text = output.get_data(as_text=True)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn("# service=git-upload-pack", output_text)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn(" refs/heads/master\n0000", output_text)
|
|
Patrick Uiterwijk |
a50651 |
|
|
Patrick Uiterwijk |
a50651 |
@patch.dict('pagure.config.config', {
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PULL_PUSH': True,
|
|
Patrick Uiterwijk |
a50651 |
'ALLOW_HTTP_PUSH': True,
|
|
Patrick Uiterwijk |
a50651 |
'HTTP_REPO_ACCESS_GITOLITE': None,
|
|
Patrick Uiterwijk |
a50651 |
})
|
|
Patrick Uiterwijk |
a50651 |
def test_http_push(self):
|
|
Patrick Uiterwijk |
a50651 |
""" Test that the HTTP push gets accepted. """
|
|
Patrick Uiterwijk |
a50651 |
output = self.app.get(
|
|
Patrick Uiterwijk |
a50651 |
'/clonetest.git/info/refs?service=git-receive-pack',
|
|
Patrick Uiterwijk |
a50651 |
environ_overrides={'REMOTE_USER': 'pingou'},
|
|
Patrick Uiterwijk |
a50651 |
)
|
|
Patrick Uiterwijk |
a50651 |
self.assertEqual(output.status_code, 200)
|
|
Patrick Uiterwijk |
a50651 |
output_text = output.get_data(as_text=True)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn("# service=git-receive-pack", output_text)
|
|
Patrick Uiterwijk |
a50651 |
self.assertIn(" refs/heads/master\x00", output_text)
|