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

There is a stored XSS in the article review area #38

Closed
1f3lse opened this issue Sep 18, 2018 · 10 comments
Closed

There is a stored XSS in the article review area #38

1f3lse opened this issue Sep 18, 2018 · 10 comments

Comments

@1f3lse
Copy link

1f3lse commented Sep 18, 2018

Comment area does not do input filtering
Poc
</p><img%20src=1%20onerror=alert(1)><p>

@94fzb
Copy link
Owner

94fzb commented Sep 19, 2018

@3lse The img tag not allow in comment, And i cat reproduce

@1f3lse
Copy link
Author

1f3lse commented Sep 19, 2018

@94fzb Need to capture the package for reproduction

@1f3lse
Copy link
Author

1f3lse commented Sep 19, 2018

@94fzb Maybe you have no restrictions on the server or storage.

@94fzb
Copy link
Owner

94fzb commented Sep 19, 2018

image

@3lse Comment use jsoup clean text, so img tag will remove when add comment

@1f3lse
Copy link
Author

1f3lse commented Sep 19, 2018

@94fzb Here is a way of attack by middlemen, by modifying traffic packets to achieve the purpose of attack.

@1f3lse
Copy link
Author

1f3lse commented Sep 19, 2018

So you need to do a verification of the received data to avoid this problem.

@94fzb
Copy link
Owner

94fzb commented Sep 19, 2018

So you need to do a verification of the received data to avoid this problem.

Verification browser received data ?

@1f3lse
Copy link
Author

1f3lse commented Sep 19, 2018

@94fzb yes!

@94fzb
Copy link
Owner

94fzb commented Sep 19, 2018

@3lse I think https is better way, and now the program support https

@1f3lse
Copy link
Author

1f3lse commented Sep 19, 2018

@94fzb Maybe

@94fzb 94fzb closed this as completed Oct 24, 2019
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

No branches or pull requests

2 participants