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

Fix link of axum-sessions in ECOSYSTEM.md #2862

Merged
merged 2 commits into from
Aug 10, 2024

Conversation

sontixyou
Copy link
Contributor

Motivation

I found that the axum-sessions crate introduced in ECOSYSTEM has been moved to tower-sessions crate.
ref: https://docs.rs/axum-sessions/latest/axum_sessions/

Solution

I replaced the link to axum-sessions crate with tower-sessions crate

ECOSYSTEM.md Outdated
@@ -28,7 +28,7 @@ If your project isn't listed here and you would like it to be, please feel free
- [aide](https://docs.rs/aide): Code-first Open API documentation generator with [axum integration](https://docs.rs/aide/latest/aide/axum/index.html).
- [axum-typed-routing](https://docs.rs/axum-typed-routing/latest/axum_typed_routing/): Statically typed routing macros with OpenAPI generation using aide.
- [axum-jsonschema](https://docs.rs/axum-jsonschema/): A `Json<T>` extractor that does JSON schema validation of requests.
- [axum-sessions](https://docs.rs/axum-sessions): Cookie-based sessions for axum via async-session.
- [tower-sessions](https://docs.rs/tower-sessions): Cookie-based sessions for axum.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like it's not just for axum, but anything based on tower + http, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I fixed description about axum.

@jplatte jplatte merged commit 52ae7bb into tokio-rs:main Aug 10, 2024
18 checks passed
@sontixyou sontixyou deleted the fix-link-in-ecosystem branch August 13, 2024 09:48
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