Skip to content
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

BUG: Requirement of Proper Input Handling in Signup form #9

Open
pand-coder opened this issue Oct 1, 2024 · 5 comments
Open

BUG: Requirement of Proper Input Handling in Signup form #9

pand-coder opened this issue Oct 1, 2024 · 5 comments

Comments

@pand-coder
Copy link

pand-coder commented Oct 1, 2024

Security Vulnerability: Client-Side Injection in Password Field

Issue Description:

The password field currently allows the input of special characters (e.g., <, >, <script>), which can lead to client-side injection attacks like Cross-Site Scripting (XSS). This poses a security risk as it allows malicious scripts to be executed on the client side.

Steps to Reproduce:

  1. Go to the login/signup form.
  2. Enter a valid email in the email field.
  3. In the password field, input a script such as <script>alert(1)</script>.
  4. Submit the form.

Expected Behavior:

Special characters such as <, >, and script tags should be disallowed in the password field to prevent injection attacks.

Actual Behavior:

Special characters are allowed, and malicious scripts can be executed.

###FIX
I would like to incorporate proper javascript Input Handling to prevent this type of client side injection or cross scripting attack
Please assign me this issue as i would like to contribute regarding this issue with respect secure aspects of coding principles

image

@Coreharshit
Copy link

i would like to work on this

@Amna-Hassan04
Copy link
Owner

@pand-coder I am assigning you the task. this app uses streamlit if you have problem implementing javascript then do disscuss with me here or a mentor. I am not really sure if you can use javascript with streamlit.

@Amna-Hassan04
Copy link
Owner

@Coreharshit I have assigned the task to pand-coder on a first come - first served basis. there are other new issues opened you can ask to get assigned to them.

@Coreharshit
Copy link

Okay !

@pand-coder
Copy link
Author

Hello @Amna-Hassan04 can i complete it by tomorrow as i need some time in understanding the code base and never worked with streamlit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants