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

CSInterface.js DOM text reinterpreted as HTML #212

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Shivam7-1
Copy link

By using innerText, it will avoid the risk of HTML injection, as these properties automatically escape any HTML special characters in the provided text. This helps prevent cross-site scripting (XSS) vulnerabilities by treating the input as plain text rather than interpreted HTML. Always be cautious when dealing with user input or dynamic content to prevent security risks.

@Shivam7-1
Copy link
Author

Hi @adamplouff Could You Please Review this PR
Thanks

@Shivam7-1
Copy link
Author

Hi @adamplouff Could You Please Review this PR
Thanks

3 similar comments
@Shivam7-1
Copy link
Author

Hi @adamplouff Could You Please Review this PR
Thanks

@Shivam7-1
Copy link
Author

Hi @adamplouff Could You Please Review this PR
Thanks

@Shivam7-1
Copy link
Author

Hi @adamplouff Could You Please Review this PR
Thanks

@Shivam7-1
Copy link
Author

Hi @adamplouff Is there Anything Else is Required From My Side
To get This PR merge
Thanks

@Shivam7-1
Copy link
Author

Hi @adamplouff Could You Please Review this PR
Thanks

1 similar comment
@Shivam7-1
Copy link
Author

Hi @adamplouff Could You Please Review this PR
Thanks

@sculli-net
Copy link

I have been using this tool for a little bit and, since I am starting to get into open source, want to make some potential contributions myself but is no one checking any of the pull requests? I really hope this tool isn't abandoned given its usefulness.

@vmedium
Copy link
Collaborator

vmedium commented Aug 27, 2024 via email

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

Successfully merging this pull request may close these issues.

3 participants