Add support for Axum framework

[brunoribeiro127]

This MR adds support for the Axum framework.

[DarrenBaldwin07]

Thank you so much for the PR @brunoribeiro127 - taking a first pass today and will come back with some review comments.

[capveg-netdebug]

Nice PR @brunoribeiro127 - what's the status of this? I'd love to use this code as I'm also working with Axum. I'm still reading through the changes but it doesn't look like you used https://docs.rs/axum-login/latest/axum_login/ - any reason?

[brunoribeiro127]

@capveg-netdebug Thanks for reviewing. To be honest, I was not aware of axum_login at all. It seems like the best way forward though.

[capveg-netdebug]

Thanks for the quick reply. If that's the case, I might try to push some of my axum_login integration here for folks to look at/think about. It might be that my use-case is a bit unique in that I want to include some non-trivial state with the user identity that's stored locally to my app and not in Clerk but I'm curious how you're thinking about it as it affects the design.

[DarrenBaldwin07]

I'd like to get this in too! I personally havent used axum much so not sure what best practices are here. I do like the idea of having a central authorizer that then gets consumed by the different web frameworks middleware setups so this seems right to me. I think maybe best approach would be to get this reviewed and merged as-is so that we can have axum support - after that, i'd be happy to take PRs for refactors/improvements

[mies]

would love to have this merged in as well, thank you both for the work @brunoribeiro127 @DarrenBaldwin07

[hazelmeow]

@DarrenBaldwin07 What would it take to get this merged? :)

[DarrenBaldwin07]

There are a couple things I want to tweak but those can be done after merging this. Address my comments (and resolve conflicts) and we can get this merged + released 🚀

Thanks again for the PR!

[DarrenBaldwin07]

This function needs to be exported - some users interface with this directly. I think its better if we revert this function back to what it was before (always returning true or false)

pub fn validate_jwt(token: &str, jwks: JwksModel) -> Result<(bool, ClerkJwt), bool> {

[brunoribeiro127]

Done.

[DarrenBaldwin07]

Lets also make changes to lib.rs to export the authorizer.rs module

[brunoribeiro127]

I've made the function public. The import path changed from use clerk_rs::validators::actix::validate_jwt to use clerk_rs::validators::authorizer::validate_jwt.