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

introduce a :paranoid flag to prevent leaking the existence of an email in the db (POC) #132

Closed
wants to merge 1 commit into from

Conversation

dkhgh
Copy link

@dkhgh dkhgh commented Jan 1, 2023

Wanted to try out a proof of concept after reading this issue.

@mikker
Copy link
Owner

mikker commented Jan 2, 2023

Thank you! This is not a bad solution 👍

I'm not sure we should add another config option for just this. Let me think about it for a bit.

@dkhgh
Copy link
Author

dkhgh commented Jan 2, 2023

I'm not sure we should add another config option for just this. Let me think about it for a bit.

Can totally relate to that. Just wanted to make a tangible so it's easier to think about. At least that's how it usually works for me.

@mikker
Copy link
Owner

mikker commented Jun 17, 2023

Closing this for now. I don't think it is necessary for Passwordless currently. Thank you for contributing though!

@mikker mikker closed this Jun 17, 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.

None yet

2 participants