-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add optional toggle for the space character #30
Comments
hi @kylehakala, so you just need a toggle for implementing space character inside your random generator? |
Yeah, something like “include spaces”
… On Oct 25, 2018, at 07:56, CapitanFindusFI ***@***.***> wrote:
hi @kylehakala, so you just need a toggle for implementing space character inside your random generator?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
@kylehakala ok, I'm on it |
@kylehakala the password-generator library that you're using doesn't include the whitespace character while listing all available chars, so I'm pretty sure it won't ever consider it while writing its random password. Anyway, I could proceed writing whitespaces at random positions inside your string, but it won't be a really good solution imho |
@CapitanFindusFI it’s not my project, so as far as I’m concerned, it could be implemented however you see appropriate. But it would probably be best to write this in a way such that the code can be used to add spaces between specific parts of the text to help advance the passphrase feature. @devshawn what say you? |
@CapitanFindusFI Did you ever jump into this? :) |
The toggle for any style of password/phrase is important due to of a lack of consistency in systems supporting spaces or no-spaces in their password requirements.
The text was updated successfully, but these errors were encountered: