You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
web should prevent to inject javascript or other possible dynamic contents.
Actual behaviour
javascript can be injected and executed from/for all space members
Solution
for now we sanitize the md rendered content and restrict those options, as a long term goal the backend also should handle those situations and strip out potential code parts.
The text was updated successfully, but these errors were encountered:
as a long term goal the backend also should handle those situations and strip out potential code parts.
I don't think that this is necessary or a good idea. You might want to sync markdown files with html img tags and so on but we don't want to display that in the spaces description. I think it makes sense to just let the frontend sanitize the content before displaying.
Steps to reproduce
[link](javascript:alert('aaa'))
as contentExpected behaviour
web should prevent to inject javascript or other possible dynamic contents.
Actual behaviour
javascript can be injected and executed from/for all space members
Solution
for now we sanitize the md rendered content and restrict those options, as a long term goal the backend also should handle those situations and strip out potential code parts.
The text was updated successfully, but these errors were encountered: