🔒 Code Review for College Project

[pure cargo]

@cunning yacht

Python help channel opened

Remember to:

• Ask your Python question, not if you can ask or if there's an expert who can help.
• Show a code sample as text (rather than a screenshot) and the error message, if you've got one.
• Explain what you expect to happen and what actually happens.

:warning: Do not pip install anything that isn't related to your question, especially if asked to over DMs.

Closes after a period of inactivity, or when you send !close.

[cunning yacht]

https://github.com/CJ-Presley/Lorem-Ipsum

in the read me there is a form for both the front and backend

if any of y'all could review them it'd be much apprieciated (if not ty anyways)

[tribal roost]

i refined your code in back end

check your issues tab

[cunning yacht]

I see, thank you very much

[tribal roost]

yw

what project is it btw?

online authentication?

[cunning yacht]

nope, full-stack development of a web solution

[tribal roost]

what web solution

[cunning yacht]

the entire repository is the solution i ended out with 🤷

its to make a site w/ functioning database , front-backend linking, etc from scratch in 30hrs (for implementation) we got 20hrs for planning tho

[edgy breach]

im having high expectations from the description /s

okay uhm

@cunning yacht why do you need cryptography if you're not using it

[cunning yacht]

no idea, I guess I got into the habit of importing it before I found a better solution

[edgy breach]

you're not even using it wdym better solution 😭

🥴

what is this code

CORS(app, resources={r"/*": {"origins": "*"}})
```*why are you using CORS if you're allowing everything anyway*

[cunning yacht]

I wasn't really sure how to use it to be honest, this was my 1st time working with a lot of this stuff

[edgy breach]

passwordPattern = (
        r"^(?=.*[0-9])(?=.*[a-z])(?=.*[A-Z])([a-zA-Z0-9@#$%^_&-+=]+){5,16}$"
    )```

don't limit a user's password length unless its at an absurd limit like 2048 chars

[cunning yacht]

oh

[edgy breach]

ok but if you see 6 nested excepts there's definitely a problem

[cunning yacht]

touche

[edgy breach]

also that wont accept a password like

abccdEFA910231!!™️©️

(at least, i don think)

or this either apparently

(after removing the 16 char limitation

ok it literally doesnt accept periods 😭

[cunning yacht]

oh dear, I have no defense other than I had to rush most of the backend stuff

[edgy breach]

lol

[cunning yacht]

tbf, this is great feedback, I now know what I shouldn't do XD

[edgy breach]

hmm

[cunning yacht]

?

[pure cargo]

Python help channel closed

This help channel has been closed and it's no longer possible to send messages here. If your question wasn't answered, feel free to create a new post in #1035199133436354600. To maximize your chances of getting a response, check out this guide on asking good questions.