Feature - Random Alphanumeric Password Generation - #10
Open
eli-johnson-04 wants to merge 10 commits into
Open
Conversation
Created flags for various options for random password generation and made the generation option a condition as opposed to defaulting to keywords.
Corrected the argparse attributes by replacing hyphens with underscores and set up the lists containing characters for the random password generation
Script now generates single random passwords of varying types and saves to the output file. Skeleton for using special characters is included and commented but not implemented.
Special characters are now an option with random password generation. Also fixed output formatting issue in line 616.
Added mutual exclusivity to the random password generation commands. Random commands do not interfere with keyword commands, so mutual exclusivity is not necessary there. Not the safest code but it works. Still need to update README and attempt to determine SystemExit: 2 error with using multiple random commands under mutual exclusion.
also added a sample random command to the help command output
Enabled the option to specify the length of a random password and included default values for all options. Also added comments.
Random passwords must be a minimum of one character in length
Owner
|
@eli-johnson-04 hey, I'll check it out! Did you close it because you found bugs or anything? I'll get back to you but i need some time |
Author
Hey! I had to contribute to an open-source project for a class, and I wasn’t 100% sure of what I was doing. I put in all due effort for the feature though! |
gaafr05
approved these changes
Mar 1, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Feature Request from @kali64amd - Added flags to generate a single random alphanumeric, uppercase alphanumeric, upper- and lowercase alphanumeric, and solely numeric password, with an optional flag for including special characters. The alphanumeric passwords default to length 12 and the numeric defaults to 9, although any number >= 1 can be included as an optional argument with the commands. The script defaults to keyword-based generation and the new commands will not interfere with using keywords in the case that both are included.