mirror of
https://github.com/cisagov/manage.get.gov.git
synced 2025-07-26 20:48:40 +02:00
add extra handling
This commit is contained in:
parent
4e0a9c0b3d
commit
7c9ca52867
3 changed files with 13 additions and 31234 deletions
31234
src/debug_response.html
31234
src/debug_response.html
File diff suppressed because one or more lines are too long
|
@ -171,6 +171,7 @@ urlpatterns = [
|
||||||
path(
|
path(
|
||||||
"admin/logout/",
|
"admin/logout/",
|
||||||
RedirectView.as_view(pattern_name="logout", permanent=False),
|
RedirectView.as_view(pattern_name="logout", permanent=False),
|
||||||
|
name="logout",
|
||||||
),
|
),
|
||||||
path(
|
path(
|
||||||
"admin/analytics/export_data_type/",
|
"admin/analytics/export_data_type/",
|
||||||
|
|
|
@ -2,6 +2,7 @@
|
||||||
Centralized permissions management for the registrar.
|
Centralized permissions management for the registrar.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
|
from django.urls import URLResolver
|
||||||
from registrar.decorators import (
|
from registrar.decorators import (
|
||||||
HAS_PORTFOLIO_DOMAIN_REQUESTS_ANY_PERM,
|
HAS_PORTFOLIO_DOMAIN_REQUESTS_ANY_PERM,
|
||||||
IS_STAFF,
|
IS_STAFF,
|
||||||
|
@ -141,6 +142,7 @@ UNCHECKED_URLS = [
|
||||||
"available",
|
"available",
|
||||||
"rdap",
|
"rdap",
|
||||||
"todo",
|
"todo",
|
||||||
|
"logout",
|
||||||
]
|
]
|
||||||
|
|
||||||
|
|
||||||
|
@ -153,12 +155,22 @@ def verify_all_urls_have_permissions():
|
||||||
|
|
||||||
resolver = get_resolver()
|
resolver = get_resolver()
|
||||||
missing_permissions = []
|
missing_permissions = []
|
||||||
|
missing_names = []
|
||||||
|
|
||||||
# Collect all URL pattern names
|
# Collect all URL pattern names
|
||||||
for pattern in resolver.url_patterns:
|
for pattern in resolver.url_patterns:
|
||||||
|
# Skip URLResolver objects (like admin.site.urls)
|
||||||
|
if isinstance(pattern, URLResolver):
|
||||||
|
continue
|
||||||
|
|
||||||
if hasattr(pattern, "name") and pattern.name:
|
if hasattr(pattern, "name") and pattern.name:
|
||||||
if pattern.name not in URL_PERMISSIONS and pattern.name not in UNCHECKED_URLS:
|
if pattern.name not in URL_PERMISSIONS and pattern.name not in UNCHECKED_URLS:
|
||||||
missing_permissions.append(pattern.name)
|
missing_permissions.append(pattern.name)
|
||||||
|
else:
|
||||||
|
raise ValueError(f"URL pattern {pattern} has no name")
|
||||||
|
|
||||||
|
if missing_names:
|
||||||
|
raise ValueError(f"The following URL patterns have no name: {missing_names}")
|
||||||
|
|
||||||
return missing_permissions
|
return missing_permissions
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue