Skip to content

Fix/fast json schema #156

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Jul 18, 2025
Merged

Fix/fast json schema #156

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e747d0e
update dependancies fastjsonschema and imports on utils_functions.py
MaxNumerique Jul 16, 2025
b980189
à faire valider.
MaxNumerique Jul 16, 2025
8ace7d2
from fastjsonschema.exceptions import JsonSchemaDefinitionException
MaxNumerique Jul 16, 2025
366fa6f
fix(schema validation): replace jssonschema with fastjsonschema
MaxNumerique Jul 17, 2025
582687f
Apply prepare changes
MaxNumerique Jul 17, 2025
9ad7016
fix(schema validation): match fastjsonschema test_utils error msg
MaxNumerique Jul 18, 2025
dccb6e3
Merge branch 'fix/fast_json_schema' of https://github.com/Geode-solut…
MaxNumerique Jul 18, 2025
4b95370
fix(fastjsonschema): add error msg logic into the test_utils.py dans…
MaxNumerique Jul 18, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion requirements.in
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ geode-simplex
geode-explicit
geode-implicit
geode-common
jsonschema
fastjsonschema
Flask[async]
Flask-Cors
werkzeug
8 changes: 3 additions & 5 deletions requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ blinker==1.9.0
# via flask
click==8.2.1
# via flask
fastjsonschema==2.21.1
# via -r requirements.in
flask[async]==3.1.1
# via
# -r requirements.in
Expand Down Expand Up @@ -60,10 +62,6 @@ itsdangerous==2.2.0
# via flask
jinja2==3.1.6
# via flask
jsonschema==4.24.0
# via -r requirements.in
jsonschema-specifications==2025.4.1
# via jsonschema
markupsafe==3.0.2
# via
# flask
Expand Down Expand Up @@ -120,4 +118,4 @@ werkzeug==3.1.3
# via
# -r requirements.in
# flask
# flask-cors
# flask-cors
21 changes: 16 additions & 5 deletions src/opengeodeweb_back/utils_functions.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@

# Third party imports
import flask
from jsonschema import validate
from jsonschema.exceptions import ValidationError
import fastjsonschema
import importlib.metadata as metadata

# Local application imports
Expand Down Expand Up @@ -82,9 +81,21 @@ def validate_request(request, schema):
json_data = {}

try:
validate(instance=json_data, schema=schema)
except ValidationError as e:
flask.abort(400, f"Validation error: {e.message}")
validate = fastjsonschema.compile(schema)
validate(json_data)
except fastjsonschema.JsonSchemaException as e:
error_msg = str(e)

if "data must contain" in error_msg:
field = error_msg.split("data must contain ['")[1].split("']")[0]
error_msg = f"'{field}' is a required property"
elif "data must not contain" in error_msg:
field = error_msg.split("data must not contain {'")[1].split("'")[0]
error_msg = (
f"Additional properties are not allowed ('{field}' was unexpected)"
)

flask.abort(400, f"Validation error: {error_msg}")


def set_interval(func, sec, args=None):
Expand Down