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 to Flask 2.3 #659

Merged
merged 18 commits into from
Aug 14, 2023
Merged

Update to Flask 2.3 #659

merged 18 commits into from
Aug 14, 2023

Conversation

lukasjuhrich
Copy link
Collaborator

@lukasjuhrich lukasjuhrich commented Aug 11, 2023

Blocker for #658

In Werkzeug 2.3, the LocalProxy implementation has been reworked
slightly; `_get_current_object` is now closure directly binding the
`local`, so setting the attribute does not do anything.
Since we call `.remove()`, we actually expect a `scoped_session`
rather than just a `Session`.  Furthermore, we make use of the newer
generic typing features of said types.
This is necessary so that `load_default` works. That's what triggered
that annoying deprecation warning about `metadata`.

See https://marshmallow.readthedocs.io/en/latest/changelog.html#id11
This is reported:
https://foss.heptapod.net/python-libs/passlib/-/issues/185

Given that passlib 1.8 will be dropping old versions[1], the
maintainer will most likely fix this in 1.8.

[1] https://foss.heptapod.net/python-libs/passlib/-/issues/119
This appearently is not necessary anymore.
the second attribute to `LocalProxy` now serves as an attribute name
to be looked up.
The `type: ignore` is probably obsolete due to the sentry-sdk update,
and the url_for false positive seems like a mypy quirk.
This contains the fix for sqlalchemyorg/sphinx-paramlinks#20,
which adresses some deprecation warnings.
in 2.1, calling `.json` on a request without the proper media type
aborts with 415 WRONG MEDIA TYPE.  `is_json` is what we want here.
Since v0.251, ruff supports `match` statements.
@lukasjuhrich lukasjuhrich merged commit b24566a into develop Aug 14, 2023
7 checks passed
@lukasjuhrich lukasjuhrich deleted the flask_23 branch September 6, 2023 19:11
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.

1 participant