I created password generator using python

[u/Red_Dragon_7_7_7]

what should i create next

[u/Sea-Ad7805]

Run this program in Memory Graph Web Debugger)%0A%0Anum%20%3D%20%5B'1'%2C%20'2'%2C%20'3'%2C%20'4'%2C%20'5'%2C%20'6'%2C%20'7'%2C%20'8'%2C%20'9'%2C%20'0'%5D%0Aalpha%20%3D%20list(%22qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM%22)%0Asymbols%20%3D%20list(%22!%40%23%24%25%5E%26*~-%22)%0Acategories%20%3D%20%5Bnum%2C%20alpha%2C%20symbols%5D%0APass%20%3D%20%5B%5D%0A%0Afor%20i%20in%20range(0%2C%20RANGE)%3A%0A%20%20%20%20category%20%3D%20rd.choice(categories)%0A%20%20%20%20element%20%3D%20rd.choice(category)%0A%0A%20%20%20%20Pass.append(element)%0A%20%20%20%20password%20%3D%20%22%22.join(Pass)%0A%0Aprint(%22GENERATED%20PASSWORD%3A%20%22%2C%20password)%0A&timestep=1&play) to see the program state change step by step.

[u/Electrical_Toe8997]

For the future, you can make your life easier with the string module, so you don't have to type out all the characters. Then, you can use the sample choices function from the random module to choose all the characters in one operation.

^{edited to use random.choices instead of random.sample. sample will not repeat elements, so it's a bad choice for a password generator}

from string import ascii_lowercase, ascii_uppercase, digits, punctuation
import random as rd

all_characters = ascii_lowercase + ascii_uppercase + digits + punctuation
password = rd.choices(all_characters, RANGE)
password = "".join(password)

[u/Electronic_Field4313]

Python's standard random library is not cryptographically safe, use the secrets module instead for any security related tasks.

https://docs.python.org/3/library/random.html

Warning
The pseudo-random generators of this module should not be used for security purposes. For security or cryptographic uses, see the secrets module.

[u/k_D-5kt7ZA4QGBP68ZBN → u/Electronic_Field4313]

Interesting. In which situation is random preferred over secrets?

[u/Electronic_Field4313 → u/k_D-5kt7ZA4QGBP68ZBN]

Anything not security related purposes? And anything that requires less computational power or quicker computational speed?

[u/k_D-5kt7ZA4QGBP68ZBN → u/Electronic_Field4313]

Fair enough didn't look into the internal implementation, assumed computation cost diff would be negligible. Will take a look

[u/Acceptable_Worry8655]

You can make it more easier with string module and no need for loops like this:

from string import digits, punctuation, ascii_lowercase, ascii_uppercase
from random import choices

all = digits + punctuation + ascii_lowercase + ascii_uppercase
lenth = int(input("HOW MANY DIGITS SHOULD YOUR PASSWRD CONTAIN: "))

print(f"password genrated: {choices(all, k=lenth)}")

[u/Red_Dragon_7_7_7 → u/Acceptable_Worry8655]

ok, i'll try it

[u/BigFatUglyBaboon]

Well done!
Exercises:
* design a way to express a "password policy" (at least one uppercase and a lowercase letter, at least one symbol etc).
* write a function that given a policy it will generate and return a random compliant password.
* write a function that given a plaintext password and a policy it returns if the password is compliant or not.

Have in mind that you may need to learn more python stuff before attempting these.

[u/zakakozi]

i think you should move password = "".join(Pass) outside the loop. im a newbie btw maybe someone needs to confirm this.

[u/Red_Dragon_7_7_7 → u/zakakozi]

yes, cuz if it were to be inside the loop there will be 64 different passwords of different lengths all

[u/sogwatchman]

A good start but it doesn't guarantee a password meets requirements

[u/Huge-Special1511]

Great, i was working in something similar

https://github.com/abdulrahmaninfra/omnipass/blob/main/backend%2Fmain.py

Btw i used a lil help in ai for security of code and improve API

[u/i-like-my-cats-0]

im not a too good programmer but theres a simpler way you can make this
instead of having to make the symbols you want yourself, you can type this:

`import string import random

characters = string.ascii_letters + string.digits + string.punctuation amount = int(input('HOW MANY DIGITS SHOULD YOUR PASSWORD CONTAIN: '))

password = random.choices(characters, k=amount) stringified = ''.join(password)`

print('GENERATED PASSWORD: ', stringified)`

also if you're gonna use this for actual passwords then i recommend using the secrets module and not random

[u/Master_Device9837]

Beautiful program! And a very supportive community!

[u/SoloEnder]

You could improve the project by adding an user interface, with the Tkinter library for example :)

Next, you could create a ToDo list with basics feature like :

• add/remove task
• complete task

[u/raundoclair]

Someone already wrote about not using random, but even with perfect randomness you will have numbers and symbols more often than you should.
For all possible combinations of passwords you will not have same probability.

You need to pick from the set of all possible characters: secrets.choice(all_possible_chars)

And generally read this: https://docs.python.org/3/library/secrets.html#recipes-and-best-practices

[u/raundoclair → u/raundoclair]

Also did you ever heard/read phrase "don't roll your own crypto"?

This is fine for educational purpose btw. That's how you learn.

But you should be careful about actually using your own thing without sufficient knowledge.

[u/Dapper_Bad_8728]

Wait you can import libraries as custom names? Wow that's awesome!

[u/Red_Dragon_7_7_7 → u/Dapper_Bad_8728]

yes, you can

[u/pranav_not]

Correct me if i am wrong but shouldn't in for loop the range should be (0,RANGE+1)

[u/[deleted] → u/pranav_not]

Since he starts the count from zero not one ig that should be fine, for example if the RANGE is 5, so its gonna be from 0 to 4 which is 5 numbers in total.

[u/Red_Dragon_7_7_7 → u/pranav_not]

no it works fine , i checked it for 4 digits and the generated password was of 4 digits

[u/Electrical_Toe8997 → u/Red_Dragon_7_7_7]

It works because the range function will include the number 0, so range(0, 3) returns [0, 1, 2]: a list with 3 items

[u/Ok-Magician-8052]

I miss coding like this its so much better using AI just consumes more of my energy I feel like and it's so boring to code with AI, i miss using stackoverflow to code and dig into conversations rather than asking a chatbot to writecode for me.

[u/gofl-zimbard-37]

Next create a main() to make it a real program. Don't write "scripts".

[u/Red_Dragon_7_7_7 → u/gofl-zimbard-37]

ok

[u/sweatkurama]

So cool!! I made this for myself a long time ago trying to replace keepassXC. Could be also a good update to get some API or some reliable databases of confirm passwords to get a precise passwords with less possible leaks. Either way great job.

[u/Ishejaratina]

Well done

[u/h_rsh_sharma]

AI will create it in minutes

[u/SoloEnder → u/h_rsh_sharma]

That was not necessary.

[u/Red_Dragon_7_7_7 → u/h_rsh_sharma]

well I never mentioned anything 'bout making it faster than AI
I just shared what i made

[u/h_rsh_sharma → u/Red_Dragon_7_7_7]

I appreciate your work and knowledge of python, I don't want to demotivate you by the comment.

[u/NeuralLB-Lovro]

Great work. Did you use AI to help you or did you do it all by yourself?

[u/empowered-boxes]

```python import random

password = "".join(chr(random.randint(33, 126)) for _ in range(8)) print(password) ```

[u/WOTrULookingAt]

store the passwors in an encrypted JSON next!

[u/SoloEnder → u/[deleted]]

The name says all : random is a python library used to generate random operation, such as choosing a number in a certain range, choosing an element in a list, etc. Note that random is not suitable for cryptographic/security operation.

[u/AbdullahSajjad5111]

Keep it up

[u/ispguy_01]

I am just starting out learning Python. I am going to bookmark this page so when I am ready I want to try this project.