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

Clear userinput #10

Open
DanielSiepmann opened this issue Jan 11, 2012 · 1 comment
Open

Clear userinput #10

DanielSiepmann opened this issue Jan 11, 2012 · 1 comment

Comments

@DanielSiepmann
Copy link
Collaborator

Currently, the input of the user is not validated or cleaned.
This should be fixed, to prevent them from attacking the bot or other uses by inserting JavaScript or something else.

Perhaps we can use http://de.php.net/manual/en/function.htmlspecialchars.php .
We should write an own method to cleanup the userinput.

@super3
Copy link
Owner

super3 commented Jan 12, 2012

We can use the HTML special chars for now.

Yeah that needs to be fixed. I have already exploited and a tried that with the previous script. I think this bot is a little better because it goes to the log file, instead of straight to a browser window like the old one.

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

2 participants