entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 37 Releases Activity

misc: abort requests if site-options is invalid (#53587) #562

Merged
fpeters merged 1 commits from wip/53587-error-on-invalid-site-options into main 2023-08-11 00:31:14 +02:00
Conversation 0 Commits 1 Files Changed 3 +13 -2
fpeters commented 2023-08-06 10:42:41 +02:00
Owner
Copy Link

C'est décalé ainsi vers _q_traverse pour permettre à l'exception ensuite d'être attrapée par le finish_failed_request normal, pour être logguée vers sentry etc.

Mais curieusement dans les tests ça ne passe cependant pas par ce chemin du code, on récupère directement l'exception.

C'est décalé ainsi vers _q_traverse pour permettre à l'exception ensuite d'être attrapée par le finish_failed_request normal, pour être logguée vers sentry etc. Mais curieusement dans les tests ça ne passe cependant pas par ce chemin du code, on récupère directement l'exception.
fpeters added 1 commit 2023-08-06 10:42:42 +02:00
fpeters changed title from WIP: misc: abort requests if site-options is invalid (#53587) to misc: abort requests if site-options is invalid (#53587) 2023-08-06 12:18:25 +02:00
tnoel approved these changes 2023-08-06 18:47:30 +02:00
wcs/root.py
@ -283,6 +283,8 @@ class RootDirectory(Directory):
get_publisher().substitutions.feed(get_request().user)
def _q_traverse(self, path):
if get_publisher().site_options_exception:
tnoel commented 2023-08-06 18:47:26 +02:00
Owner
Copy Link

J'aurais écrit « is not None: » ici mais ça marchera bien ainsi.

J'aurais écrit « is not None: » ici mais ça marchera bien ainsi.
fpeters merged commit 78f7292cfa into main 2023-08-11 00:31:14 +02:00
fpeters deleted branch wip/53587-error-on-invalid-site-options 2023-08-11 00:31:14 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#562
No description provided.
Delete Branch "wip/53587-error-on-invalid-site-options"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?