-
-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[Rust] Bump axum and axum-extra #20548
Conversation
Thanks for the PR but your commit (as shown in the Commits tab) is not linked to your Github account, which means this PR won't count as your contribution in https://github.com/OpenAPITools/openapi-generator/graphs/contributors. Let me know if you need help fixing it. |
cc @linxGnu for review |
@wing328 Fixed. Thanks for the catch! |
@whirm Basically, LGTM, the same as I did last month (my local branch): master...linxGnu:openapi-generator:rust_axum_authorization However, please wait for #20463 to merge first. It contains necessary changes. |
modules/openapi-generator/src/main/resources/rust-axum/Cargo.mustache
Outdated
Show resolved
Hide resolved
@whirm please resolve the merge conflicts when you've time. |
- This bump requires fixing path parameters due to matchit transitive dependency changes (See tokio-rs/axum#2645). - The `Host` extractor has been moved from `axum` to `axum-extra`. - Axum now ships multipart extractor support, switch to that.
d17a31f
to
6e4495f
Compare
Done! |
This bump required fixing path parameters due to
matchit
transitive dependencychanges (See chore: update matchit to 0.8 tokio-rs/axum#2645).
The
Host
extractor has been moved fromaxum
toaxum-extra
.PR checklist
Commit all changed files.
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*
.IMPORTANT: Do NOT purge/delete any folders/files (e.g. tests) when regenerating the samples as manually written tests may be removed.
master
(upcoming7.x.0
minor release - breaking changes with fallbacks),8.0.x
(breaking changes without fallbacks)@frol, @farcaller, @richardwhiuk, @paladinzh, @jacob-pro ping!