Skip to content

update to axum v0.7 #4

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

Merged
merged 5 commits into from
Dec 31, 2024
Merged

update to axum v0.7 #4

merged 5 commits into from
Dec 31, 2024

Conversation

utterstep
Copy link
Contributor

Hi! First of all, thank you for your lib!

This PR updates the axum dependency to v0.7 – what do you think about it? 0.7 was released a bit more than a year ago and the there is already RC for 0.8.

@cmackenzie1
Copy link
Owner

Thanks @utterstep, this looks great. I think we need to bump hyper as well to 1.0 and it should be good to merge!

@utterstep
Copy link
Contributor Author

Thank you for the feedback! I updated the example to use axum::serve to keep it simple, is that OK?

It can be found in c92620a

I also noticed that some of the dependencies were used only indirectly, by other dependencies (like hyper, tower, serde_json). I took the liberty of removing them in the separate commit. Also, I moved chrono to dev-dependencies as it is only the example code that depends on it, and not the lib.
This was done in the 7873c88.

And lastly I updated reqwest and jsonwebtoken to the latest versions also in bfa0b6d.

I understand that last two commits are out of the scope of the PR so in case you'd like to keep to the original purpose – I have no problems reverting any of them or both.

@cmackenzie1 cmackenzie1 self-requested a review December 31, 2024 17:52
@cmackenzie1
Copy link
Owner

Looks great, thanks @utterstep !

@cmackenzie1 cmackenzie1 merged commit 47cac6f into cmackenzie1:main Dec 31, 2024
1 check passed
@utterstep utterstep deleted the axum-0.7 branch December 31, 2024 19:52
@utterstep utterstep restored the axum-0.7 branch December 31, 2024 19:52
@cmackenzie1
Copy link
Owner

Version 0.2.0 has been published with these changes!

https://github.com/cmackenzie1/axum-jwt-auth/releases/tag/v0.2.0
https://crates.io/crates/axum-jwt-auth/0.2.0
https://docs.rs/crate/axum-jwt-auth/0.2.0

@utterstep utterstep deleted the axum-0.7 branch January 2, 2025 08:05
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