fix permission bug

4dn-dcic · Aug 11, 2023 · ff5e646 · ff5e646
1 parent 9622440
commit ff5e646
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -6,6 +6,12 @@ snovault
 Change Log
 ----------
 
+10.0.2
+======
+
+* Repair bug in ``permission`` implementation involving restricted fields
+
+
 10.0.1
 ======
 

diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "dcicsnovault"
-version = "10.0.1"
+version = "10.0.2"
 description = "Storage support for 4DN Data Portals."
 authors = ["4DN-DCIC Team <[email protected]>"]
 license = "MIT"

diff --git a/snovault/schema_utils.py b/snovault/schema_utils.py
@@ -277,6 +277,10 @@ class IgnoreUnchanged(ValidationError):
     pass
 
 
+class SnoPermissionError(ValidationError):
+    pass
+
+
 def requestMethod(validator, requestMethod, instance, schema):
     ignored(instance, schema)
     if validator.is_type(requestMethod, "string"):
@@ -300,7 +304,7 @@ def permission(validator, permission, instance, schema):
     context = request.context
     if not request.has_permission(permission, context):
         error = "permission %r required" % permission
-        yield IgnoreUnchanged(error)
+        yield SnoPermissionError(error)
 
 
 VALIDATOR_REGISTRY = {}