Skip to content
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.

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

Update Flask JSON interfaces #145

Merged
merged 3 commits into from
Oct 11, 2023
Merged

Update Flask JSON interfaces #145

merged 3 commits into from
Oct 11, 2023

Conversation

thomas-bc
Copy link
Collaborator

Originating Project/Creator
Affected Component
Affected Architectures(s) Flask JSON
Related Issue(s) nasa/fprime#2077
Has Unit Tests (y/n)
Builds Without Errors (y/n)
Unit Tests Pass (y/n)
Documentation Included (y/n)

Change Description

Updates Flask JSON interfaces to the newer pattern.

Rationale

Flask deprecated and removed the older way of doing things. See nasa/fprime#2077

From Flask release notes:

Setting custom json_encoder and json_decoder classes on the app or a blueprint, and the corresponding json.JSONEncoder and JSONDecoder classes, are deprecated. JSON behavior can now be overridden using the app.json provider interface. pallets/flask#4692

@thomas-bc thomas-bc requested a review from LeStarch October 3, 2023 01:01
@thomas-bc
Copy link
Collaborator Author

Would allow to resolve the Dependabot alert on a Flask vulnerability: https://github.com/nasa/fprime/security/dependabot/11

@LeStarch LeStarch merged commit 6a545e0 into nasa:devel Oct 11, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants