-
Notifications
You must be signed in to change notification settings - Fork 34
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
HTML input is not sanitized #36
Comments
Where and how you can use any HTML? |
In the Device nickname field |
True, seems that the readme is still valid :)
|
is this issue mentioned only reflecting to client side java script or even to server side? |
I didn't check the server part, however I am able to display javascript
alerts or open popups on a friends pc using snapweb like this.
nanderer ***@***.***> schrieb am Mo., 10. Jan. 2022, 00:31:
… is this issue mentioned only reflecting to client side java script or even
to server side?
—
Reply to this email directly, view it on GitHub
<#36 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMGSKD5BOJQHKT6NVHQENF3UVILGDANCNFSM5IMGDBEA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
The server doesn't interpret any JavaScript, it just serves files within the doc root directory |
Fixed in v0.6.0 |
You can use any HTML as and can even inject javascript by using
<img src="broken" onerror="your_js_code"></img>
. If many users are using snapweb, this could be considered a security issue.The text was updated successfully, but these errors were encountered: