Skip to content

Commit 6595af5

Browse files
committed
fix(backend): add "doc" to forbidden names
1 parent 21da01b commit 6595af5

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

docat/docat/utils.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ def is_forbidden_project_name(name: str) -> bool:
126126
a page on the docat website.
127127
"""
128128
name = name.lower().strip()
129-
return name in ["upload", "claim", "delete", "help"]
129+
return name in ["upload", "claim", "delete", "help", "doc"]
130130

131131

132132
UNITS_MAPPING = [

docat/tests/test_rename.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ def test_rename_rejects_forbidden_project_name(client_with_claimed_project):
8686
assert create_response.status_code == 201
8787

8888
with patch("os.rename") as rename_mock:
89-
for project_name in ["upload", "claim", "Delete ", "help"]:
89+
for project_name in ["upload", "claim", "Delete ", "help", "Doc"]:
9090
rename_response = client_with_claimed_project.put(f"/api/some-project/rename/{project_name}", headers={"Docat-Api-Key": "1234"})
9191
assert rename_response.status_code == 400
9292
assert rename_response.json() == {

docat/tests/test_upload.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ def test_upload_rejects_forbidden_project_name(client_with_claimed_project):
160160
"""
161161

162162
with patch("docat.app.remove_docs") as remove_mock:
163-
for project_name in ["upload", "claim", " Delete ", "help"]:
163+
for project_name in ["upload", "claim", " Delete ", "help", "DOC"]:
164164
response = client_with_claimed_project.post(
165165
f"/api/{project_name}/1.0.0", files={"file": ("index.html", io.BytesIO(b"<h1>Hello World</h1>"), "plain/text")}
166166
)

0 commit comments

Comments
 (0)