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

New example: session_auth_axum #589

Merged
merged 6 commits into from
Mar 7, 2023
Merged

New example: session_auth_axum #589

merged 6 commits into from
Mar 7, 2023

Conversation

j0lol
Copy link
Contributor

@j0lol j0lol commented Feb 27, 2023

New example using axum_sessions_auth for managing user authentication, based on the todo example.

@j0lol j0lol changed the title New example: todo_login_app_sqlite_axum New example: session_auth_axum Feb 27, 2023
@gbj
Copy link
Collaborator

gbj commented Feb 28, 2023

Thanks — I added it to our CI testing and I had to fix a few imports in main.rs to get it running.

On first load I find that I get

Server Error: error running server function: Pool missing.

If I've done something like log in, or log out, or navigate to another page and back, I don't get this... But if I reload, I do on first SSR. I wonder if it's some kind of a race condition where it's not being provided initially?

@j0lol
Copy link
Contributor Author

j0lol commented Feb 28, 2023

thats odd? i'm not sure what's wrong..

@j0lol
Copy link
Contributor Author

j0lol commented Feb 28, 2023

I should probably add errorboundraries to the rest of the forms...

@gbj
Copy link
Collaborator

gbj commented Mar 2, 2023

I think there are some fixes you made that I don't think you've pushed yet. Were there any other changes you wanted to make?

@j0lol
Copy link
Contributor Author

j0lol commented Mar 4, 2023

missed the comment, ill merge my fixes now :)

@j0lol
Copy link
Contributor Author

j0lol commented Mar 4, 2023

all good now!

@gbj
Copy link
Collaborator

gbj commented Mar 7, 2023

Thanks — looks good!

@gbj gbj merged commit 369ea85 into leptos-rs:main Mar 7, 2023
@j0lol j0lol deleted the login-example branch March 7, 2023 10:15
gbj pushed a commit that referenced this pull request Mar 21, 2023
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