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
Would it be beyond the scope of this plugin to escape the attributes passed to Image.generateHTML?
Currently unescaped attributes result in mangled HTML output:
Image.generateHTML(stats,{alt: 'Some "unescaped" user content',});
<!-- Output --><imgsrc="/img.png" alt="Some "unescaped" user content" />
This is pretty simple to solve by dropping in an attribute escaping library of one’s choice before passing attributes to generateHTML, but I wondered if it would be a good idea to do this by default? Are there any attributes that generateHTMLshouldn’t escape?
The text was updated successfully, but these errors were encountered:
Would it be beyond the scope of this plugin to escape the attributes passed to
Image.generateHTML
?Currently unescaped attributes result in mangled HTML output:
This is pretty simple to solve by dropping in an attribute escaping library of one’s choice before passing attributes to
generateHTML
, but I wondered if it would be a good idea to do this by default? Are there any attributes thatgenerateHTML
shouldn’t escape?The text was updated successfully, but these errors were encountered: