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 axum integration version #1106

Merged
merged 4 commits into from Nov 25, 2022

Conversation

tomerze
Copy link
Contributor

@tomerze tomerze commented Oct 9, 2022

Makes our axum integration work with axum 0.6.* which introduced some breaking changes.

Currently axum `0.6.*` is not fully released, this commit will need
to be updated to target the full release version.
@tomerze
Copy link
Contributor Author

tomerze commented Oct 9, 2022

We should probably wait until axum 0.6.* is fully released before we merge this.

@sunli829
Copy link
Collaborator

Thanks, it will be merged when Axum 0.6 is released. 🙂

In axum 0.6.* `FromRequest` always consumes the request's body. `FromRequestParts` was
also introduced for when you don't need to consume the body.

You can read about it more in axum's changelog.
We need some way to document how to use this crate for users who are yet
to upgrade to axum `0.6.*`. The idea here is that when we publish the crate
again we will increase it's version making `4.0.15` the last version
supporting axum `0.5.*`.
@dspicher
Copy link
Contributor

axum 0.6.0 has been released 🚀

@sunli829 sunli829 merged commit daec444 into async-graphql:master Nov 25, 2022
@sunli829
Copy link
Collaborator

Thanks! 🙂

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.

None yet

3 participants