Skip to content
Snippets Groups Projects
Verified Commit 9e509793 authored by Andreas Ellewsen's avatar Andreas Ellewsen
Browse files

Remove trailing slash from api paths

A proxy refused to include trailing slashes, and so they have been
removed.

Resolves: GREG-76
parent 7bf658fb
No related branches found
No related tags found
No related merge requests found
Pipeline #96619 passed
......@@ -26,31 +26,31 @@ urlpatterns = router.urls
# Allowing trailing slashes to be optional for the URLs below
urlpatterns += [
re_path(
r"^persons/(?P<person_id>[0-9]+)/roles/?$",
r"^persons/(?P<person_id>[0-9]+)/roles$",
RoleViewSet.as_view({"get": "list", "post": "create"}),
name="person_role-list",
),
re_path(
r"^persons/(?P<person_id>[0-9]+)/roles/(?P<id>[0-9]+)/?$",
r"^persons/(?P<person_id>[0-9]+)/roles/(?P<id>[0-9]+)$",
RoleViewSet.as_view(
{"get": "retrieve", "patch": "partial_update", "delete": "destroy"}
),
name="person_role-detail",
),
re_path(
r"^persons/(?P<person_id>[0-9]+)/identities/?$",
r"^persons/(?P<person_id>[0-9]+)/identities$",
IdentityViewSet.as_view({"get": "list", "post": "create"}),
name="person_identity-list",
),
re_path(
r"^persons/(?P<person_id>[0-9]+)/identities/(?P<id>[0-9]+)/?$",
r"^persons/(?P<person_id>[0-9]+)/identities/(?P<id>[0-9]+)$",
IdentityViewSet.as_view(
{"get": "retrieve", "delete": "destroy", "patch": "partial_update"}
),
name="person_identity-detail",
),
re_path(
r"^sponsors/(?P<sponsor_id>[0-9]+)/guests/?$",
r"^sponsors/(?P<sponsor_id>[0-9]+)/guests$",
SponsorGuestsViewSet.as_view({"get": "list"}),
name="sponsor_guests-list",
),
......
......@@ -3,6 +3,6 @@ import pytest
@pytest.mark.django_db
def test_health_says_ok(client):
response = client.get("/api/health/")
response = client.get("/api/health")
assert response.content == b"OK"
assert response.status_code == 200
......@@ -24,17 +24,17 @@ from greg.api.views.health import Health
urlpatterns: List[URLResolver] = [
path(
"schema/",
"schema",
SpectacularAPIView.as_view(versioning_class=NamespaceVersioning),
name="schema",
), # type: ignore
path(
"schema/swagger-ui/",
"schema/swagger-ui",
SpectacularSwaggerView.as_view(
url_name="schema", versioning_class=NamespaceVersioning
), # type: ignore
name="swagger-ui",
),
path("api/health/", Health.as_view()), # type: ignore
path("api/health", Health.as_view()), # type: ignore
path("api/v1/", include((api_urls.urlpatterns, "greg"), namespace="v1")),
]
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment