[codecommit] fix repository name check
This commit is contained in:
parent
cb1eb79b89
commit
cba1b2e180
@ -6,13 +6,23 @@ from .models import codecommit_backends
|
|||||||
from .exceptions import InvalidRepositoryNameException
|
from .exceptions import InvalidRepositoryNameException
|
||||||
|
|
||||||
|
|
||||||
|
def _is_repository_name_valid(repository_name):
|
||||||
|
name_regex = re.compile(r"[\w\.-]+")
|
||||||
|
result = name_regex.split(repository_name)
|
||||||
|
if len(result) > 0:
|
||||||
|
for match in result:
|
||||||
|
if len(match) > 0:
|
||||||
|
return False
|
||||||
|
return True
|
||||||
|
|
||||||
|
|
||||||
class CodeCommitResponse(BaseResponse):
|
class CodeCommitResponse(BaseResponse):
|
||||||
@property
|
@property
|
||||||
def codecommit_backend(self):
|
def codecommit_backend(self):
|
||||||
return codecommit_backends[self.region]
|
return codecommit_backends[self.region]
|
||||||
|
|
||||||
def create_repository(self):
|
def create_repository(self):
|
||||||
if not self._is_repository_name_valid(self._get_param("repositoryName")):
|
if not _is_repository_name_valid(self._get_param("repositoryName")):
|
||||||
raise InvalidRepositoryNameException()
|
raise InvalidRepositoryNameException()
|
||||||
|
|
||||||
repository_metadata = self.codecommit_backend.create_repository(
|
repository_metadata = self.codecommit_backend.create_repository(
|
||||||
@ -24,7 +34,7 @@ class CodeCommitResponse(BaseResponse):
|
|||||||
return json.dumps({"repositoryMetadata": repository_metadata})
|
return json.dumps({"repositoryMetadata": repository_metadata})
|
||||||
|
|
||||||
def get_repository(self):
|
def get_repository(self):
|
||||||
if not self._is_repository_name_valid(self._get_param("repositoryName")):
|
if not _is_repository_name_valid(self._get_param("repositoryName")):
|
||||||
raise InvalidRepositoryNameException()
|
raise InvalidRepositoryNameException()
|
||||||
|
|
||||||
repository_metadata = self.codecommit_backend.get_repository(
|
repository_metadata = self.codecommit_backend.get_repository(
|
||||||
@ -34,7 +44,7 @@ class CodeCommitResponse(BaseResponse):
|
|||||||
return json.dumps({"repositoryMetadata": repository_metadata})
|
return json.dumps({"repositoryMetadata": repository_metadata})
|
||||||
|
|
||||||
def delete_repository(self):
|
def delete_repository(self):
|
||||||
if not self._is_repository_name_valid(self._get_param("repositoryName")):
|
if not _is_repository_name_valid(self._get_param("repositoryName")):
|
||||||
raise InvalidRepositoryNameException()
|
raise InvalidRepositoryNameException()
|
||||||
|
|
||||||
repository_id = self.codecommit_backend.delete_repository(
|
repository_id = self.codecommit_backend.delete_repository(
|
||||||
@ -45,8 +55,3 @@ class CodeCommitResponse(BaseResponse):
|
|||||||
return json.dumps({"repositoryId": repository_id})
|
return json.dumps({"repositoryId": repository_id})
|
||||||
|
|
||||||
return json.dumps({})
|
return json.dumps({})
|
||||||
|
|
||||||
def _is_repository_name_valid(self, repository_name):
|
|
||||||
name_regex = re.compile(r"[\w\.-]+")
|
|
||||||
result = name_regex.fullmatch(repository_name)
|
|
||||||
return result
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user