Wagtail permissions and url fix (#2975)

ChristopherChudzicki · web-flow · commit 68b56f8bd1c5 · 2025-09-30T16:07:23.000-04:00
diff --git a/cms/wagtail_api/views.py b/cms/wagtail_api/views.py
@@ -54,7 +54,7 @@ def get_permissions(self):
         """
         Returns the appropriate permissions based on the 'type' query parameter.
         """
-        page_type = self.request.query_params.get("type")
+        page_type = self.request.query_params.get("type", "").lower()
         if page_type in PageType.anonymous_access_allowed_types():
             return [AllowAny()]
         return [IsAuthenticated()]
diff --git a/openapi/hooks.py b/openapi/hooks.py
@@ -169,24 +169,24 @@ class WrappedPagesRetrieveView(WagtailPagesRetrieveSchemaView):
     )
     endpoints.append(
         (
-            "/api/v2/pages/?fields=*&type=cms.CertificatePage",
-            "^api/v2/pages/?fields=*&type=cms.CertificatePage$",
+            "/api/v2/pages/?fields=*&type=cms.certificatepage",
+            "^api/v2/pages/?fields=*&type=cms.certificatepage$",
             "GET",
             certificate_view,
         )
     )
     endpoints.append(
         (
-            "/api/v2/pages/?fields=*&type=cms.CoursePage",
-            "^api/v2/pages/?fields=*&type=cms.CoursePage$",
+            "/api/v2/pages/?fields=*&type=cms.coursepage",
+            "^api/v2/pages/?fields=*&type=cms.coursepage$",
             "GET",
             course_view,
         )
     )
     endpoints.append(
         (
-            "/api/v2/pages/?fields=*&type=cms.ProgramPage",
-            "^api/v2/pages/?fields=*&type=cms.ProgramPage$",
+            "/api/v2/pages/?fields=*&type=cms.programpage",
+            "^api/v2/pages/?fields=*&type=cms.programpage$",
             "GET",
             program_view,
         )
diff --git a/openapi/specs/v0.yaml b/openapi/specs/v0.yaml
@@ -1130,9 +1130,9 @@ paths:
                 - $ref: '#/components/schemas/CertificatePage'
                 - $ref: '#/components/schemas/Page'
           description: Returns a page of any known Wagtail page type
-  /api/v2/pages/?fields=*&type=cms.CertificatePage:
+  /api/v2/pages/?fields=*&type=cms.certificatepage:
     get:
-      operationId: pages_?fields=*&type=cms.CertificatePage_retrieve
+      operationId: pages_?fields=*&type=cms.certificatepage_retrieve
       description: Returns pages of type cms.CertificatePage
       summary: List all Certificate Pages
       tags:
@@ -1144,9 +1144,9 @@ paths:
               schema:
                 $ref: '#/components/schemas/CertificatePageList'
           description: ''
-  /api/v2/pages/?fields=*&type=cms.CoursePage:
+  /api/v2/pages/?fields=*&type=cms.coursepage:
     get:
-      operationId: pages_?fields=*&type=cms.CoursePage_retrieve
+      operationId: pages_?fields=*&type=cms.coursepage_retrieve
       description: Returns pages of type cms.CoursePage
       summary: List all Course Pages
       parameters:
@@ -1164,9 +1164,9 @@ paths:
               schema:
                 $ref: '#/components/schemas/CoursePageList'
           description: ''
-  /api/v2/pages/?fields=*&type=cms.ProgramPage:
+  /api/v2/pages/?fields=*&type=cms.programpage:
     get:
-      operationId: pages_?fields=*&type=cms.ProgramPage_retrieve
+      operationId: pages_?fields=*&type=cms.programpage_retrieve
       description: Returns pages of type cms.ProgramPage
       summary: List all Program Pages
       parameters:
diff --git a/openapi/specs/v1.yaml b/openapi/specs/v1.yaml
@@ -1130,9 +1130,9 @@ paths:
                 - $ref: '#/components/schemas/CertificatePage'
                 - $ref: '#/components/schemas/Page'
           description: Returns a page of any known Wagtail page type
-  /api/v2/pages/?fields=*&type=cms.CertificatePage:
+  /api/v2/pages/?fields=*&type=cms.certificatepage:
     get:
-      operationId: pages_?fields=*&type=cms.CertificatePage_retrieve
+      operationId: pages_?fields=*&type=cms.certificatepage_retrieve
       description: Returns pages of type cms.CertificatePage
       summary: List all Certificate Pages
       tags:
@@ -1144,9 +1144,9 @@ paths:
               schema:
                 $ref: '#/components/schemas/CertificatePageList'
           description: ''
-  /api/v2/pages/?fields=*&type=cms.CoursePage:
+  /api/v2/pages/?fields=*&type=cms.coursepage:
     get:
-      operationId: pages_?fields=*&type=cms.CoursePage_retrieve
+      operationId: pages_?fields=*&type=cms.coursepage_retrieve
       description: Returns pages of type cms.CoursePage
       summary: List all Course Pages
       parameters:
@@ -1164,9 +1164,9 @@ paths:
               schema:
                 $ref: '#/components/schemas/CoursePageList'
           description: ''
-  /api/v2/pages/?fields=*&type=cms.ProgramPage:
+  /api/v2/pages/?fields=*&type=cms.programpage:
     get:
-      operationId: pages_?fields=*&type=cms.ProgramPage_retrieve
+      operationId: pages_?fields=*&type=cms.programpage_retrieve
       description: Returns pages of type cms.ProgramPage
       summary: List all Program Pages
       parameters:
diff --git a/openapi/specs/v2.yaml b/openapi/specs/v2.yaml
@@ -1130,9 +1130,9 @@ paths:
                 - $ref: '#/components/schemas/CertificatePage'
                 - $ref: '#/components/schemas/Page'
           description: Returns a page of any known Wagtail page type
-  /api/v2/pages/?fields=*&type=cms.CertificatePage:
+  /api/v2/pages/?fields=*&type=cms.certificatepage:
     get:
-      operationId: pages_?fields=*&type=cms.CertificatePage_retrieve
+      operationId: pages_?fields=*&type=cms.certificatepage_retrieve
       description: Returns pages of type cms.CertificatePage
       summary: List all Certificate Pages
       tags:
@@ -1144,9 +1144,9 @@ paths:
               schema:
                 $ref: '#/components/schemas/CertificatePageList'
           description: ''
-  /api/v2/pages/?fields=*&type=cms.CoursePage:
+  /api/v2/pages/?fields=*&type=cms.coursepage:
     get:
-      operationId: pages_?fields=*&type=cms.CoursePage_retrieve
+      operationId: pages_?fields=*&type=cms.coursepage_retrieve
       description: Returns pages of type cms.CoursePage
       summary: List all Course Pages
       parameters:
@@ -1164,9 +1164,9 @@ paths:
               schema:
                 $ref: '#/components/schemas/CoursePageList'
           description: ''
-  /api/v2/pages/?fields=*&type=cms.ProgramPage:
+  /api/v2/pages/?fields=*&type=cms.programpage:
     get:
-      operationId: pages_?fields=*&type=cms.ProgramPage_retrieve
+      operationId: pages_?fields=*&type=cms.programpage_retrieve
       description: Returns pages of type cms.ProgramPage
       summary: List all Program Pages
       parameters:

Original file line number	Diff line number	Diff line change
`@@ -169,24 +169,24 @@ class WrappedPagesRetrieveView(WagtailPagesRetrieveSchemaView):`
`169`	`169`	`)`
`170`	`170`	`endpoints.append(`
`171`	`171`	`(`
`172`		`- "/api/v2/pages/?fields=*&type=cms.CertificatePage",`
`173`		`- "^api/v2/pages/?fields=*&type=cms.CertificatePage$",`
	`172`	`+ "/api/v2/pages/?fields=*&type=cms.certificatepage",`
	`173`	`+ "^api/v2/pages/?fields=*&type=cms.certificatepage$",`
`174`	`174`	`"GET",`
`175`	`175`	`certificate_view,`
`176`	`176`	`)`
`177`	`177`	`)`
`178`	`178`	`endpoints.append(`
`179`	`179`	`(`
`180`		`- "/api/v2/pages/?fields=*&type=cms.CoursePage",`
`181`		`- "^api/v2/pages/?fields=*&type=cms.CoursePage$",`
	`180`	`+ "/api/v2/pages/?fields=*&type=cms.coursepage",`
	`181`	`+ "^api/v2/pages/?fields=*&type=cms.coursepage$",`
`182`	`182`	`"GET",`
`183`	`183`	`course_view,`
`184`	`184`	`)`
`185`	`185`	`)`
`186`	`186`	`endpoints.append(`
`187`	`187`	`(`
`188`		`- "/api/v2/pages/?fields=*&type=cms.ProgramPage",`
`189`		`- "^api/v2/pages/?fields=*&type=cms.ProgramPage$",`
	`188`	`+ "/api/v2/pages/?fields=*&type=cms.programpage",`
	`189`	`+ "^api/v2/pages/?fields=*&type=cms.programpage$",`
`190`	`190`	`"GET",`
`191`	`191`	`program_view,`
`192`	`192`	`)`