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
Hello @darylldoyle thank you for this great library, it works really well.
I would like to use this library as a validation when uploading a SVG. In general it works ok, but there are some "feature requests" which probably could improve the handling. The use case it, that we want to decide if we allow the upload of the SCG or not not to clean up the uploaded SVG.
So we tried to sanitize and then check if there are any xmlIssues or if sanitize returns false. So far so good, but we stumbled here over some challenges:
If there are PHP opening/closing tags, the sanitizer cleans them, but it's never reported somewhere
If there is a comment in the SVG (which is ok I guess) it's reported as xmlIssuehere
There is no differentiation between the errors a comment raises the same issue level as a javascript
It's hard to determinate what error occured just from having the strings (e.g. to output then custom error messages)
It would be nice to enhance the doc block comments with the description of the parameters and return values.
Is there anything which is on the roadmap for future releases?
Thank you in advance.
The text was updated successfully, but these errors were encountered:
Hello @darylldoyle thank you for this great library, it works really well.
I would like to use this library as a validation when uploading a SVG. In general it works ok, but there are some "feature requests" which probably could improve the handling. The use case it, that we want to decide if we allow the upload of the SCG or not not to clean up the uploaded SVG.
So we tried to sanitize and then check if there are any
xmlIssues
or if sanitize returns false. So far so good, but we stumbled here over some challenges:xmlIssue
hereIt would be nice to enhance the doc block comments with the description of the parameters and return values.
Is there anything which is on the roadmap for future releases?
Thank you in advance.
The text was updated successfully, but these errors were encountered: