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

fixes issue #243 #244

Closed
wants to merge 1 commit into from
Closed

fixes issue #243 #244

wants to merge 1 commit into from

Conversation

duarme
Copy link
Contributor

@duarme duarme commented Jan 22, 2025

Adds a reminder to clean up expired sessions into README.md.

@mikker
Copy link
Owner

mikker commented Jan 28, 2025

I don't think this is a problem for the gem. Thank you for contributing but I don't agree that we need to include this.

@mikker mikker closed this Jan 28, 2025
@duarme
Copy link
Contributor Author

duarme commented Jan 28, 2025

I've been using passwordless for years and realized I had the database full of expired sessions only a few days ago. A heads-up in the README would have been helpful in my case. Anyway, no problem, I still like this, a lot.

@mikker
Copy link
Owner

mikker commented Jan 28, 2025

It's not really a problem unless someone feels it's a problem for them and when they decide it is, it's relatively easy to clean up manually.

Thank you!

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