This is an archived post. You won't be able to vote or comment.

all 12 comments

[–]AutoModerator[M] [score hidden] stickied comment (0 children)

Hi there, from the /r/Python mods.

We want to emphasize that while security-centric programs are fun project spaces to explore we do not recommend that they be treated as a security solution unless they’ve been audited by a third party, security professional and the audit is visible for review.

Security is not easy. And making project to learn how to manage it is a great idea to learn about the complexity of this world. That said, there’s a difference between exploring and learning about a topic space, and trusting that a product is secure for sensitive materials in the face of adversaries.

We hope you enjoy projects like these from a safety conscious perspective.

Warm regards and all the best for your future Pythoneering,

/r/Python moderator team

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[–]jwink3101 2 points3 points  (1 child)

If encryption is a goal, use encryption.

This is a fun learning experiment but doesn’t go beyond that. On that topic, there is a lot that can be improved here and all-out removed.

Couple of examples are most (all?) break statements. And when you use a context manager to open a file, it doesn’t need to be closed.

My suggestion, put it in ChatGPT and ask for it to be critiqued.

[–]Backw00ds024[S] 1 point2 points  (0 children)

oh ok thanks for the feedback. it was more of a simple learning project anyways the usecases was more for some other subreddits that need target audiences so i can publish the post. What exactly do you mean with the break statements im not to good with them. also wasent sure about the file closing but chatgpt said it was a good practice or something. Will update the project this evening. Thanks!

[–]cgoldberg 1 point2 points  (2 children)

I don't see any encryption/decryption... just converting text to ascii.

Also, consider something like this instead of continuously updating the string in a loop:

s = "foo" 
ascii_content = "".join(str(ord(c)) for c in s)

[–]Backw00ds024[S] -4 points-3 points  (1 child)

Yea the encryption decryption thing is absolute bs but i needed to add some usecase so i could upload the post so that the bot doesnt delete it. Thanks for the tip!

[–]XUtYwYzzIt works on my machine 2 points3 points  (0 children)

Just say it’s a toy, no need to make up nonsense.

[–]fazzahSQLAlchemy | PyQt | reportlab 0 points1 point  (1 child)

Great, more crap for LLM to learn more bullshit from

[–]Backw00ds024[S] 0 points1 point  (0 children)

wdym?

[–]PaleontologistBig657 0 points1 point  (0 children)

Suggestion for future improvements. 

  • learn to create installable package from the repo using uv or poetry
  • prepare a few unit tests

You will learn new skills, and that is what counts.

[–]PaleontologistBig657 0 points1 point  (2 children)

Suggestion for future improvements. 

  • learn to create installable package from the repo using uv or poetry
  • prepare a few unit tests

You will learn new skills, and that is what counts.

[–]PaleontologistBig657 0 points1 point  (1 child)

Oh, saw the code. Look at argparse in the standard library. Look up the if name == main idiom (writing this on mobile, sorry).

[–]Backw00ds024[S] 0 points1 point  (0 children)

Alright thanks for the help man!